-
Notifications
You must be signed in to change notification settings - Fork 1
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
Response missing "obs" data, returning last cached response #5
Comments
Now seeing additional error:
Which repeats for each of the configured sensors. |
The first issue happens when the WeatherFlow Tempest REST API fails to respond - it's a networking error.
You can always trying manually calling the API and confirming your station responds with the correct JSON structure. There should be a top level key called curl https://swd.weatherflow.com/swd/rest/observations/station/{insert your station id} --header "Authorization: Bearer {insert your token}" As for your second exception I'm not sure yet, I'll take a look. |
Hello Chase, Thanks for making this plugin available and the detailed response. In my home automation setup, I believe that #3 is the most likely for producing the error. My thoughts are:
My current theory is that with two different WeatherFlow apps polling the WeatherFlow server there is a pretty good chance that they could be sampling at the same time resulting in one app being rejected. I am in the process of verifying this theory and understand best way to address. Will get back to you with my findings. Regards, |
I had this error. I found the token had a trailing space and that was enough to reject the request and cause the obs missing error. Check that your token is copied exactly. Working fine ever since. KevinSent from my iPadOn Feb 27, 2023, at 12:02 AM, Doug B ***@***.***> wrote:
Hello Chase,
Thanks for making this plugin available and the detailed response.
In my home automation setup, I believe that #3 is the most likely for producing the error. My thoughts are:
I have no indiction from the Tempest app that the PWS is going off line.
I checked that the suggested curl provided the expected JSON response for the obs key.
I do, however, also have have an instance of Home-Assistant running with the WeatherFlow integration.
My current theory is that with two different WeatherFlow apps polling the WeatherFlow server there is a pretty good chance that they could be sampling at the same time resulting in one app being rejected.
I am in the process of verifying this theory and understand best way to address. Will get back to you with my findings.
Regards,
Doug
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: ***@***.***>
|
@kskenyon, thanks for the suggestion. Checked and the token has no trailing space. The error occurrence is pretty random and the Tempest data as viewed through homebridge or HomeKit is correct. |
Hello Chase, I have spent quite a bit of time trying to understand these errors. I agree with you that the The second I also have a homebridge plugin ( Regards, |
I believe I have been able to address and found additional error in the process. Will be submitting a Pull Request this week for your consideration. Regards, Doug |
@DMBlakeley thanks looking forward to reviewing it! |
@chasenicholl I have submitted a change request to address the runtime errors. Changes are summarized in the CHANGE.MD file which I created. The second error noted above is now being trapped. I am not 100% sure of the root cause and my changes may trigger some thought on your part. A possible issue is that Something else that needs to be addressed is that deleted Accessories are not removed if you delete or rename in the config file. Looking forward to your feedback. Regards, Doug |
Describe Your Problem:
Several times per day I receive the error shown in the logs below. The plugin is configured as a Child Bridge and starts without issue. Data being reported seems correct. Would like to understand how to eliminate this error.
I also have a Home-Assistant plugin also polling the Weatherflow-Tempest data.
Logs:
Plugin Config:
Screenshots:
Environment:
The text was updated successfully, but these errors were encountered: