search

avwx-rest / avwx-api

REST API for parsing aviation weather data
https://avwx.rest
MIT License
71 stars 89 forks source link

Server is Down! #11

Closed CelticParser closed 5 years ago

CelticParser commented 5 years ago

Looks like you reached your quota on Azure. We would be gladly pay for an api key if you stop using the free Azure server(?).

Thnx1

devdupont commented 5 years ago

It’s definitely not on a free server anymore 😅

I’m not seeing service interruptions on the Azure ide with few 5xx errors. It’s likely there is an error in the report string that is causing the parser to fail. Could you double check the report from another service and post it here. Likely, it’s already logged an error that I’ll fix tonight, but just in case. Try https://aviationweather.gov

CelticParser commented 5 years ago

Strange this was working all day but then suddenly was getting a

Internal Server Error Server got itself in trouble

via the app I'm working on and in the browser.

A direct query on Aviation Weather's api works fine. Aviation Weather .gov query

I haven't used the AviationsWeather API because I have been getting CORS issues 😞 using axios

devdupont commented 5 years ago

Oh. Is this a coordinate lookup query?

CelticParser commented 5 years ago

Yes it is - Should have stated that to begin with.

More info: Calling using the station identifier was still working during the outage. However querying the Lat/Long for nearly an hour did not work but is suddenly now working?????

Let me know what you find. It looks like you might be using a third party API and not dirrectly via the "horses mouth?? LOL

Thx ahead of time for your findings.

devdupont commented 5 years ago

Not an API or network error. It was a parsing error like I thought. The requested METAR returned by the coord lookup looked similar to:

KGUS 180138Z 31007KT 7/8SM R23/5000VP6000FT BR VV002 01/01 A3001 RMK AO2A SLP171 SLP ESTMD $

Just as an example because this bug affected at least two reports. The error came when trying to translate sea-level pressure. It worked for "SLP171" but raised an error for "SLP" with no digits with it. The parser has been updated to check for this.