Skip to content

update authz logic#63

Merged
SantanM merged 4 commits into
developfrom
SantanM/fix-datasetid-condition
Apr 16, 2025
Merged

update authz logic#63
SantanM merged 4 commits into
developfrom
SantanM/fix-datasetid-condition

Conversation

@SantanM

@SantanM SantanM commented Mar 10, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Add logic for new role STUDY_WRITE_DQD_RESEARCHER
  • Refactor and fix the scope check to ensure user has all scopes as required by the definition of the endpoint in functions/package.json file
@SantanM SantanM requested a review from p-hoffmann as a code owner March 10, 2025 11:34
@SantanM SantanM changed the title change requireDatasetId logic: use endpoint scopes and researcher scopes update authz logic Apr 7, 2025
@jerome-ng

jerome-ng commented Apr 15, 2025

Copy link
Copy Markdown
Contributor

I think this is a good scenario for tests. I am trying to add some.
@SantanM SantanM enabled auto-merge (squash) April 15, 2025 02:11
@SantanM SantanM merged commit 62c0a11 into develop Apr 16, 2025
@SantanM SantanM deleted the SantanM/fix-datasetid-condition branch April 16, 2025 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants

@SantanM @jerome-ng @p-hoffmann @satish-a0