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

Optionnally provide request HTTP headers to processors #1899

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vprivat-ads
Copy link
Contributor

Overview

Related Issue / discussion

This PR allows to forward request http headers to processors.

Our use case is to pass an API key to a processor.

Additional information

Dependency policy (RFC2)

  • I have ensured that this PR meets RFC2 requirements

Updates to public demo

Contributions and licensing

(as per https://github.com/geopython/pygeoapi/blob/master/CONTRIBUTING.md#contributions-and-licensing)

  • I'd like to contribute feature X to pygeoapi. I confirm that my contributions to pygeoapi will be compatible with the pygeoapi license guidelines at the time of contribution
  • I have already previously agreed to the pygeoapi Contributions and Licensing Guidelines

@tomkralidis tomkralidis self-requested a review January 13, 2025 11:54
@tomkralidis tomkralidis added the OGC API - Processes OGC API - Processes label Jan 13, 2025
@tomkralidis tomkralidis added this to the 0.20.0 milestone Jan 13, 2025
webb-ben added a commit to webb-ben/pygeoapi that referenced this pull request Jan 15, 2025
webb-ben added a commit to webb-ben/pygeometa that referenced this pull request Jan 15, 2025
@vprivat-ads vprivat-ads force-pushed the process-request-headers branch 3 times, most recently from b9b06a7 to d8cc2ca Compare January 21, 2025 14:09
@vprivat-ads vprivat-ads force-pushed the process-request-headers branch from d8cc2ca to d351f30 Compare January 28, 2025 15:26
@vprivat-ads
Copy link
Contributor Author

@tomkralidis @webb-ben Should I rebase my work on top of #1902?

tomkralidis pushed a commit that referenced this pull request Jan 29, 2025
* Add outputs to OAProc

* Use **kwargs

cc #1899

* Use named function arguments
webb-ben added a commit to webb-ben/pygeometa that referenced this pull request Jan 30, 2025
tomkralidis pushed a commit to geopython/pygeometa that referenced this pull request Jan 30, 2025
* Add outputs kwargs to OAProc

* use **kwargs

cc geopython/pygeoapi#1899

* Fix typo

* Use named process arguments

* Fix missed kwarg
@vprivat-ads vprivat-ads force-pushed the process-request-headers branch 3 times, most recently from 51c7adf to ad8c02d Compare January 30, 2025 17:35
@vprivat-ads vprivat-ads force-pushed the process-request-headers branch from ad8c02d to 21e1452 Compare January 30, 2025 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OGC API - Processes OGC API - Processes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants