Allow upload and delete to /tmp #1544
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1544 +/- ##
==========================================
+ Coverage 75.32% 77.05% +1.73%
==========================================
Files 609 611 +2
Lines 45922 45991 +69
Branches 799 799
==========================================
+ Hits 34589 35440 +851
+ Misses 11245 10463 -782
Partials 88 88
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
| return unless authorization('admin') | ||
| # Anywhere other than config/SCOPE/targets_modified or config/SCOPE/tmp requires admin | ||
| if !(params[:bucket] == 'OPENC3_CONFIG_BUCKET' && (key_split[1] == 'targets_modified' || key_split[1] == 'tmp')) | ||
| return false unless authorization('admin') |
There was a problem hiding this comment.
If return value matters then it should be explicit at the end of the method.
There was a problem hiding this comment.
We're short cutting this method because we don't want to delete unless they are admin
| @@ -208,7 +208,7 @@ def delete_bucket_item(params) | |||
|
|
|||
| def delete_volume_item(params) | |||
| # Deleting requires admin | |||
| return unless authorization('admin') | |||
| return false unless authorization('admin') | |||
There was a problem hiding this comment.
If return value matters then it should be explicit at the end of the method.
|
closes #1531