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

David/check header #294

Merged
merged 3 commits into from
Nov 1, 2024
Merged

David/check header #294

merged 3 commits into from
Nov 1, 2024

Conversation

recalcitrantsupplant
Copy link
Collaborator

@recalcitrantsupplant recalcitrantsupplant commented Nov 1, 2024

Checks headers for a specified header with a specified value, can be used for e.g. API Subscriptions.
Updates Pyoxigraph to fix unrelated test failure with OGC Features API - Lawson is separately investigating.

Copy link
Collaborator

@lalewis1 lalewis1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me.
tested locally and it does what it says on the box.
tests passing, but I will investigate further.

@lalewis1 lalewis1 merged commit f43a0dc into main Nov 1, 2024
1 check passed
@lalewis1 lalewis1 mentioned this pull request Nov 1, 2024
2 tasks
avillar added a commit to avillar/prez that referenced this pull request Nov 5, 2024
commit 5d54b33
Author: recalcitrantsupplant <dcchabgood@gmail.com>
Date:   Mon Nov 4 20:36:53 2024 +1000

    Set proxy_headers to False by default as a more secure option, set forwarded-allow-ips to a sensible default of 127.0.0.1 as it is dependent on proxy_headers being set to True.
    Match Dockerfile defaults to config.py settings.

commit b2cac71
Merge: 420a038 f43a0dc
Author: recalcitrantsupplant <dcchabgood@gmail.com>
Date:   Mon Nov 4 19:58:23 2024 +1000

    Merge remote-tracking branch 'refs/remotes/origin/main' into root-path-fixes

commit f43a0dc
Merge: 2a17d2c 6d815b2
Author: Lawson Lewis <151585421+lalewis1@users.noreply.github.com>
Date:   Fri Nov 1 13:07:11 2024 +1000

    Merge pull request RDFLib#294 from RDFLib/david/check-header-clean

    David/check header

commit 6d815b2
Author: recalcitrantsupplant <dcchabgood@gmail.com>
Date:   Fri Nov 1 12:41:25 2024 +1000

    switch back to middleware as function and remove commented code.

commit e2b17b4
Author: recalcitrantsupplant <dcchabgood@gmail.com>
Date:   Fri Nov 1 11:59:40 2024 +1000

    chore: Update pyoxigraph

commit 54da1f9
Author: recalcitrantsupplant <dcchabgood@gmail.com>
Date:   Fri Nov 1 10:48:35 2024 +1000

    feat: Add ability for prez to check for a header on all requests such as an API key.

commit 2a17d2c
Author: Lawson Lewis <151585421+lalewis1@users.noreply.github.com>
Date:   Fri Nov 1 10:50:08 2024 +1000

    extend default search predicates (RDFLib#292)

    extended to include alt and hidden labels as per the recommendation in
    RDFLib#282

commit 420a038
Author: Alejandro Villar <avillar@ogc.org>
Date:   Mon Oct 28 14:45:18 2024 +0100

    Fix root_path for subapps

commit 596a768
Author: Alejandro Villar <avillar@ogc.org>
Date:   Mon Oct 28 13:19:13 2024 +0100

    Add --root-path to docker entrypoint

commit cae67ae
Author: Alejandro Villar <avillar@ogc.org>
Date:   Mon Oct 28 13:18:53 2024 +0100

    Remove root_path from request.url.path before ops

commit 5a94425
Author: Alejandro Villar <avillar@ogc.org>
Date:   Mon Oct 28 11:43:58 2024 +0100

    Enable proxy headers in uvicorn
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.

2 participants