-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Request WeatherAPI #64
Comments
Further to the discussion, could you post a screenshot of the developer console with an OWM API key in place, also the docker logs for your jump container? |
I hope this is the right info you needed ( the api key is un place ) . Docker logs for your jump container : NOTICE: PHP message: PHP Warning: fopen(/var/www/cache/application/_sites%2Fstatus%2Fhttpsauthservicepublicfrrealmsservicepublicprotocolopenidconnectauthresponsetypecodeclientidspclientscopeaddress20phone20openid20profile20emailstate7iHt2oLzfRIvG6lMYPiwzByTRgXqpedvnEnk3ZllFIA3Dredirecturihttpswwwservicepublicfropenidconnectloginnonce547KAR7mtOaVbtrM0ZdZonimpJPv4C4btfKzfHVpncI/_8e66ccfa196b812261ae510e21038fcd): Failed to open stream: Filename too long in /var/www/html/vendor/nette/caching/src/Caching/Storages/FileStorage.php on line 136 There are more logs but the seems to be all the same .. |
And this works without the API key? The errors in your logs seem unrelated to problems with the weather API. I'm reading this on my phone so can't quite see it properly but it looks like the site status checker function is caching a URL that is at the end of a redirect from one of your sites, it is complaining about the filename of the cache file created. This might be a bug but it isn't in any way related to the weather API. Could you try with the API key in place and the |
Also try without the cache bypass option but set Thanks 😀 |
This is the only logs I could see unfortunately and the issue is happening from different browsers and different machines in my LAN including from my other iDevice, however when removing the OWAPI key the page just works . |
Thanks, can you test as I asked above with cachebypass and then checkstatus? |
p/s: I have done one by one by the way , following your instructions |
Yes as docker environment variables, thanks |
Can you forward the logs with each of these in place? |
CACHEBYPASS = true logs :
██ ██ ██ ██ ██ ██ ██ ██ https://github.com/daledavies/jump
[04-Oct-2022 06:28:03] NOTICE: fpm is running, pid 23 CHECKSTATUS = false logs :
██ ██ ██ ██ ██ ██ ██ ██ https://github.com/daledavies/jump
[04-Oct-2022 06:34:23] NOTICE: fpm is running, pid 22 |
Hi There , Any feedback on this ? |
The most recent error logs you forwarded don't actually show any errors, I'm afraid there isn't much for me to work with. If you manage to capture any errors in the console or the docker error logs please forward them with detail of the configuration options in place at the time. |
I was having the same 400 response issue from the weather API too. I added a container name to the docker compose block, and also changed the environment variables mappings from the example to suite what my other docker image blocks looked like: From:
To:
I also changed the port map because my other container is 8123 already. Weather API works fine now 🤷 |
As there is no workable information in the error logs and I can't recreate the problem I'm closing this issue. If you do manage to capture any errors relating to this in your logs please reopen. |
Discussed in #63
Originally posted by Tataukee October 3, 2022
Can you add weatherapi ? The openweather has never been working for me ?
The text was updated successfully, but these errors were encountered: