Skip to content
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

EPA AQS URLs causing error #90

Closed
zmoon opened this issue Oct 25, 2022 · 9 comments
Closed

EPA AQS URLs causing error #90

zmoon opened this issue Oct 25, 2022 · 9 comments
Labels
bug Something isn't working

Comments

@zmoon
Copy link
Member

zmoon commented Oct 25, 2022

  • EPA AQS URLs erroring in link check apparently due to their server not supporting OpenSSL 3 client
    • e.g.
      (tutorial/aqs_pams: line 27) broken https://aqs.epa.gov/aqsweb/airdata/download_files.html - HTTPSConnectionPool(host='aqs.epa.gov', port=443): Max retries exceeded with url: /aqsweb/airdata/download_files.html (Caused by SSLError(SSLError(1, '[SSL: UNSAFE_LEGACY_RENEGOTIATION_DISABLED] unsafe legacy renegotiation disabled (_ssl.c:1129)')))
    • seems this is affecting the AQS test as well (232e0e4):
      FAILED tests/test_aqs.py::test_aqs - requests.exceptions.SSLError: HTTPSConnectionPool(host='aqs.epa.gov', port=443): Max retries exceeded with url: /aqsweb/airdata/daily_PM10SPEC_2019.zip (Caused by SSLError(SSLError(1, '[SSL: UNSAFE_LEGACY_RENEGOTIATION_DISABLED] unsafe legacy renegotiation disabled (_ssl.c:1131)')))

Originally posted by @zmoon in #85 (comment)

@zmoon zmoon added the bug Something isn't working label Oct 25, 2022
@zmoon zmoon changed the title EPA AWS URLs causing error EPA AQS URLs causing error Oct 25, 2022
@zmoon
Copy link
Member Author

zmoon commented Oct 25, 2022

Some more info here

@zmoon
Copy link
Member Author

zmoon commented Oct 25, 2022

Pinning openssl is an option I suppose (1.1.1q is the current < 3 release)

@zmoon
Copy link
Member Author

zmoon commented Nov 7, 2022

Also for requests.get could try this method

@bbakernoaa
Copy link
Member

@zmoon is this finished?

@zmoon
Copy link
Member Author

zmoon commented Apr 6, 2023

Still working around it in CI. AFAIK as I know the situation hasn't changed. Did you have any luck contacting them about it?

@zmoon
Copy link
Member Author

zmoon commented May 9, 2023

Just tested this locally, aqs.epa.gov URLs still have the same issue in link check and the test when using OpenSSL v3.

@zmoon
Copy link
Member Author

zmoon commented Sep 20, 2023

In CI run for #125 today, the AQS test didn't fail with OpenSSL 3.1.2.

@bbakernoaa
Copy link
Member

bbakernoaa commented Sep 20, 2023 via email

@zmoon
Copy link
Member Author

zmoon commented Sep 25, 2023

Phil Dickerson from EPA says the server is officially fixed as of last week.

Will keep this issue open until PR removing the related extra CI cases etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants