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

Clarify: Send token to discovery endpoints #416

Closed
m-mohr opened this issue Oct 18, 2021 · 3 comments · Fixed by #417
Closed

Clarify: Send token to discovery endpoints #416

m-mohr opened this issue Oct 18, 2021 · 3 comments · Fixed by #417
Assignees
Labels
data discovery documentation patch requires a patch-version (x.x.1 for example)
Milestone

Comments

@m-mohr
Copy link
Member

m-mohr commented Oct 18, 2021

Discovery endpoints (e.g. GET /collections) allow to be requested with and without Bearer Token.
Sending the Bearer Tokens with the clients is important once you have it so that private data (e.g. Collections) can be requested.
This is important for e.g. exposing Planet data through openEO Platform.

Clarify in the API that clients should send tokens if available. This will likely result in issues for all clients, none supports this yet AFAIK.

@m-mohr m-mohr added data discovery documentation patch requires a patch-version (x.x.1 for example) labels Oct 18, 2021
@m-mohr m-mohr added this to the 1.2.0 milestone Oct 18, 2021
@m-mohr m-mohr self-assigned this Oct 18, 2021
@m-mohr
Copy link
Member Author

m-mohr commented Oct 18, 2021

@flahn
Copy link
Member

flahn commented Oct 22, 2021

The openeo r client already supports this feature for processes and collections. It was requested during openEO to allow internal staff to access the full data catalogue and offer published collections to external users.

@m-mohr
Copy link
Member Author

m-mohr commented Oct 22, 2021

@flahn This is great news. Is there any caching implemented in the R client that would need to be refreshed on authentication changes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data discovery documentation patch requires a patch-version (x.x.1 for example)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants