MMM-MealieMenu Fetch Error

sdetweil

@WallysWellies I am a mealie user too.

code looks good. will have to setup and test.

it does get the data frequently
60 seconds by default…

WallysWellies

Thanks Sam. Any pointers would be good. It’s a fairly minor thing but it just seems a bit odd that after a single fetch error it just gives up forever (until restarted).

I’ve discovered the pm2 log so I will try and remember to have a look at that next time it throws the error.

I changed the config to update once every hour when I initially set it up:

updateInterval: 60 * 60

sdetweil

@WallysWellies ok… I’ll watch it running here at 15 min updates…

well, I found it…

in the node helper he sets a flag, to say request is active

if the flag is set when the request comes from the module side
he just returns… does nothing.

on error he does not clear the flag
(bad network, host, authorization… places where there is no recovery
those make sense…)

      .catch((error) => {   //<--- had and error here ... 
        this.sendSocketNotification("MEALIE_ERROR", {
          error: "FETCH_ERROR",
          details: error,
          identifier: config.identifier
        });
      })
      .finally(() => {
        this.outstandingRequest = false;   // < --- only clears flag on success
      });

one could examine the error and reset if its a potentially recoverable error…

sdetweil

@WallysWellies actually, the unrecoverable config errors are handled in a different place…

so, just copy that one line before the sendSocketNotification for error

WallysWellies

@sdetweil Thank you for your advice and apologies for the delayed response - I need to enable email notifications or something…

I will try your suggestion. Much appreciated.

WallysWellies

@sdetweil I’m looking at the node_helper.js file and reading your comments and I’m a little unsure what you are suggesting.

Are you saying I could edit this code block:

.catch((error) => {
        this.sendSocketNotification("MEALIE_ERROR", {
          error: "FETCH_ERROR",
          details: error,
          identifier: config.identifier
        });
      })

to this:

.catch((error) => {
        this.outstandingRequest = false; 
        this.sendSocketNotification("MEALIE_ERROR", {
          error: "FETCH_ERROR",
          details: error,
          identifier: config.identifier
        });
      })

?

Sorry, I can fumble my way around code but I’m far from a programmer!

sdetweil

@WallysWellies yes, correct

WallysWellies

@sdetweil Done!

I shall try and remember to report back if it seems to have worked - if I do not then we can assume I am a happy customer.

Cheers for your time.

sdetweil

@WallysWellies remember that this is not an official change, so it could be lost.if you refresh the module

zanix

@sdetweil said in MMM-MealieMenu Fetch Error:

@WallysWellies I am a mealie user too.

code looks good. will have to setup and test.

it does get the data frequently
60 seconds by default…

I should probably set the default to something higher. I had it set to 60 seconds while I was developing it.

@sdetweil said in MMM-MealieMenu Fetch Error:

@WallysWellies ok… I’ll watch it running here at 15 min updates…

well, I found it…

in the node helper he sets a flag, to say request is active

if the flag is set when the request comes from the module side
he just returns… does nothing.

on error he does not clear the flag
(bad network, host, authorization… places where there is no recovery
those make sense…)
      .catch((error) => {   //<--- had and error here ... 
        this.sendSocketNotification("MEALIE_ERROR", {
          error: "FETCH_ERROR",
          details: error,
          identifier: config.identifier
        });
      })
      .finally(() => {
        this.outstandingRequest = false;   // < --- only clears flag on success
      });
one could examine the error and reset if its a potentially recoverable error…

I thought finally() runs even after a catch(), dangit.

@WallysWellies said in MMM-MealieMenu Fetch Error:

@sdetweil I’m looking at the node_helper.js file and reading your comments and I’m a little unsure what you are suggesting.

Are you saying I could edit this code block:
.catch((error) => {
        this.sendSocketNotification("MEALIE_ERROR", {
          error: "FETCH_ERROR",
          details: error,
          identifier: config.identifier
        });
      })
to this:
.catch((error) => {
        this.outstandingRequest = false; 
        this.sendSocketNotification("MEALIE_ERROR", {
          error: "FETCH_ERROR",
          details: error,
          identifier: config.identifier
        });
      })
?

Sorry, I can fumble my way around code but I’m far from a programmer!

I will get this fix added and set the default updateInterval to something a bit more sane soon. Thanks to all for troubleshooting!