MagicMirror icon indicating copy to clipboard operation
MagicMirror copied to clipboard

[Bug] Default Weather Module - envcanada provider is no longer pulling data from Environment Canada

Open Crazylegstoo opened this issue 4 months ago • 3 comments

Environment

MagicMirror² version: 2.33.0 Node version: 22.18.0 npm version: 10.9.3 Platform: Raspberry Pi 4 - Bookworm

Which start option are you using?

node --run start

Are you using PM2?

Yes

Module

weather

Have you tried disabling other modules?

  • [x] Yes
  • [ ] No

Have you searched if someone else has already reported the issue on the forum or in the issues?

  • [x] Yes

What did you do?

Validated there are no issues in logs and that Config.js is correct. The problem is definitely specific to the envcanada provider in the Weather module

What did you expect to happen?

Verify that Environment Canada provider is not working properly

What actually happened?

Verified that Environment Canada data is not providing data to Weather module

Additional comments

The envcanada provider in the default Weather module was fixed in MM v2.33.0 to use a new URL hierarchy that Environment Canada implemented to access weather data for Canadian locations. Subsequent to this provider update, Environment Canada has implemented one further update to their URL hierarchy to make it easier to access 'current day' weather data. Here are the old and new URL formats with the updated in bold:

Old: https://dd.weather.gc.ca/citypage_weather/...

New: https://dd.weather.gc.ca/**today/**citypage_weather/...

As a result of this additional URL update, the envcanada provider is, once again, not able to pull weather data. The net effect is that the Weather module will display outdated information or will display 'loading' in place of weather data.

Solution: The envcanada provider code will need to be updated (again) to incorporate the URL format change.

Participation

  • [x] I am willing to submit a pull request for this change.

Crazylegstoo avatar Oct 10 '25 17:10 Crazylegstoo

@Crazylegstoo I don't see anyone working on the problem. Are you preparing a PR? 🙂

KristjanESPERANTO avatar Oct 18 '25 22:10 KristjanESPERANTO

We have 70 days til next release and they have broken the api two sep times. I vote for patience

sdetweil avatar Oct 18 '25 22:10 sdetweil

@Crazylegstoo I don't see anyone working on the problem. Are you preparing a PR? 🙂

Yup! Got a bit sidetracked this week but I'm about to prep a PR. The coding change is minor and has been running on my MM for a week or so without issue. I've also bundled in a small update to catch a windspeed value of 'calm' and translate it to a numeric value.

Crazylegstoo avatar Oct 18 '25 23:10 Crazylegstoo