Skip to content

Fix HHNK url#481

Merged
dbrakenhoff merged 2 commits intodevfrom
fix_hhnk
Jun 16, 2025
Merged

Fix HHNK url#481
dbrakenhoff merged 2 commits intodevfrom
fix_hhnk

Conversation

@dbrakenhoff
Copy link
Collaborator

  • Note that we are not sure whether this new layer contains the same waterlevel information as the previous one. This is something we will have to check with HHNK. Progress on that will be posted in issue HHNK data not reachable #465.

- Note that we are not sure whether this new layer contains the same waterlevel information as the previous one. This is something we will have to check with HHNK. Progress on that will be posted in issue #465.
@github-project-automation github-project-automation bot moved this to Todo in NHFLO Jun 16, 2025
@dbrakenhoff dbrakenhoff requested a review from rubencalje June 16, 2025 10:45
@dbrakenhoff dbrakenhoff linked an issue Jun 16, 2025 that may be closed by this pull request
…s defined

- assumes config[waterboard][verify] is a url for an intermediate certificate
- this certificate is downloaded and bundled with certifi and stored in a temporary file.
- this file is passed on to requests.get()
- this allows us to download HHNK data again
@github-project-automation github-project-automation bot moved this from Todo to In Progress in NHFLO Jun 16, 2025
@dbrakenhoff dbrakenhoff merged commit 47df6c7 into dev Jun 16, 2025
4 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in NHFLO Jun 16, 2025
@dbrakenhoff dbrakenhoff deleted the fix_hhnk branch June 16, 2025 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HHNK data not reachable

2 participants