Skip to content

update owslib.util.http_post to configure redirect behaviour - #1046

Merged
tomkralidis merged 1 commit into
masterfrom
allow_redirects
Aug 17, 2026
Merged

update owslib.util.http_post to configure redirect behaviour#1046
tomkralidis merged 1 commit into
masterfrom
allow_redirects

Conversation

@tomkralidis

Copy link
Copy Markdown
Member

This PR add an allow_redirects argument to owslib.util.http_post to enable callers to configure accordingly.

@tomkralidis
tomkralidis requested a review from a team August 17, 2026 03:23
@geographika

Copy link
Copy Markdown
Contributor

I presume this is to allow users to disable redirects (as the requests default is True)?

Notes to self: https://requests.readthedocs.io/en/latest/user/quickstart/#redirection-and-history

https://requests.readthedocs.io/en/latest/api/#main-interface:
allow_redirects (bool) – (optional) Boolean. Enable/disable GET/OPTIONS/POST/PUT/PATCH/DELETE/HEAD redirection. Defaults to True.

@tomkralidis

Copy link
Copy Markdown
Member Author

I presume this is to allow users to disable redirects (as the requests default is True)?

Notes to self: https://requests.readthedocs.io/en/latest/user/quickstart/#redirection-and-history

https://requests.readthedocs.io/en/latest/api/#main-interface: allow_redirects (bool) – (optional) Boolean. Enable/disable GET/OPTIONS/POST/PUT/PATCH/DELETE/HEAD redirection. Defaults to True.

Yes, in the context of providing downstream applications safeguarding (example is pycsw harveesting).

@tomkralidis
tomkralidis merged commit 395d549 into master Aug 17, 2026
4 checks passed
@tomkralidis
tomkralidis deleted the allow_redirects branch August 17, 2026 10:08
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