Skip to content

Add authz check to update a request status at creation#85

Merged
paulineribeyre merged 1 commit into
masterfrom
create-status-authz
May 26, 2026
Merged

Add authz check to update a request status at creation#85
paulineribeyre merged 1 commit into
masterfrom
create-status-authz

Conversation

@paulineribeyre

Copy link
Copy Markdown
Collaborator

Link to JIRA ticket if there is one:

New Features

Breaking Changes

Bug Fixes

Improvements

  • Add authorization check when a request's status is set to one of the UPDATE_ACCESS_STATUSES at creation

Dependency updates

Deployment changes

@github-actions

Copy link
Copy Markdown

The style in this PR agrees with black. ✔️

This formatting comment was generated automatically by a script in uc-cdis/wool.

@Avantol13 Avantol13 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, nice catch

@github-actions

Copy link
Copy Markdown

Integration Tests

filepath passed SUBTOTAL
tests/test_client_credentials.py 1 1
tests/test_requestor.py 5 5
TOTAL 6 6

Please find the detailed integration test report here

Please find the Github Action logs here

@paulineribeyre paulineribeyre merged commit 5851ef9 into master May 26, 2026
10 checks passed
@paulineribeyre paulineribeyre deleted the create-status-authz branch May 26, 2026 17:16
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