Skip to content

PUT /bucket: temporarily remove checkElevatedUser middleware#313

Merged
norrisng-bc merged 1 commit intomasterfrom
chore/put-bucket-temp-workaround
Jul 23, 2025
Merged

PUT /bucket: temporarily remove checkElevatedUser middleware#313
norrisng-bc merged 1 commit intomasterfrom
chore/put-bucket-temp-workaround

Conversation

@norrisng-bc
Copy link
Contributor

Description

Temporarily remove checkElevatedUser middleware from PUT /bucket (create bucket) endpoint.

This middleware was added in #307; other endpoints where this was added remain unchanged and still use it.

This is a temporarily workaround for SHOWCASE-4028.

Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

Middleware will be restored once a full fix is implemented in SHOWCASE-4030.

@github-actions
Copy link

Coverage Report

Totals Coverage
Statements: 55.53% ( 3057 / 5505 )
Methods: 45.73% ( 332 / 726 )
Lines: 62.14% ( 1833 / 2950 )
Branches: 48.77% ( 892 / 1829 )

@norrisng-bc norrisng-bc merged commit f7c0f1d into master Jul 23, 2025
14 of 17 checks passed
@norrisng-bc norrisng-bc deleted the chore/put-bucket-temp-workaround branch July 23, 2025 23:34
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