Skip to content
This repository has been archived by the owner on May 18, 2024. It is now read-only.

Scoreboard is stuck on "off day" with no last or next game data displayed #439

Open
tomcrugnale opened this issue Jan 3, 2024 · 1 comment

Comments

@tomcrugnale
Copy link

Just after start of 2024, my scoreboard became stuck on the "off day" screen.
My preferred team's record was showing, but only dashes were present for the last and next game data.

After turning on verbose debug logging, I noticed that this popped up when attempting to fetch the list of active games:

24-01-02 20:30:20 DEBUG send_request_headers.started request=<Request [b'GET']>
24-01-02 20:30:20 DEBUG send_request_headers.complete
24-01-02 20:30:20 DEBUG send_request_body.started request=<Request [b'GET']>
24-01-02 20:30:20 DEBUG send_request_body.complete
24-01-02 20:30:20 DEBUG receive_response_headers.started request=<Request [b'GET']>
24-01-02 20:30:20 DEBUG receive_response_headers.complete return_value=(b'HTTP/1.1', 404, b'Not Found', [(b'Date', b'Wed, 03 Jan 2024 01:30:20 GMT'), (b'Content-Type', b'text/html;charset=iso-8859-1'), (b'Transfer-Encoding', b'chunked'), (b'Connection', b'keep-alive'), (b'Cache-Control', b'must-revalidate,no-cache,no-store'), (b'Cteonnt-Length', b'379'), (b'CF-Cache-Status', b'MISS'), (b'Vary', b'Accept-Encoding'), (b'Server', b'cloudflare'), (b'CF-RAY', b'83f779d67aae36a6-YYZ'), (b'Content-Encoding', b'gzip')])
24-01-02 20:30:20 INFO HTTP Request: GET https://api-web.nhle.com/v1/score/2024-1-2 "HTTP/1.1 404 Not Found"

That URL appears to be malformed, and needs to have a leading '0' tacked on to the front of the day and month if they are less than 10.

For example, the above URL will result in a 404 response, but https://api-web.nhle.com/v1/score/2024-01-02 will return the correct game score data.

This was observed on the version 1.7.0 of the software.

@falkyre
Copy link
Contributor

falkyre commented Jan 4, 2024

For a fix you can do yourself, see this pull request should fix the issue.

#440

There's no eta on when this will be merged into a release.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants