Skip to content

Make project context header optional in all but create endpoints#370

Draft
eleftherioszisis wants to merge 4 commits intomainfrom
optional-context
Draft

Make project context header optional in all but create endpoints#370
eleftherioszisis wants to merge 4 commits intomainfrom
optional-context

Conversation

@eleftherioszisis
Copy link
Contributor

@eleftherioszisis eleftherioszisis commented Sep 19, 2025

  • Create endpoints require a mandatory vlab/proj id pair as before
  • In all other endpoints:
    • If a user passes vlab/proj id pair in the header things will work as they used to constraining the queries into that project.
    • If a user passes just their token, then the user_project_ids in the token will be used in the queries.
    • vlab/proj id context has precedence over user_project_ids

Note: Using the token is more permissive because the query will check the authorized_project_id of the entities against all authorized user project ids.

@jdcourcol
Copy link
Contributor

Create endpoints require a mandatory vlab/proj id pair as before

@eleftherioszisis why not just the project_id ?

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