-
Notifications
You must be signed in to change notification settings - Fork 48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/cleversafe url #744
Feat/cleversafe url #744
Conversation
The style in this PR agrees with This formatting comment was generated automatically by a script in uc-cdis/wool. |
This pull request introduces 1 alert when merging 81a1dea into f5a712b - view on LGTM.com new alerts:
|
This pull request fixes 15 alerts when merging 0c6a1d7 into f5a712b - view on LGTM.com fixed alerts:
|
Pull Request Test Coverage Report for Build 8551
💛 - Coveralls |
…dpoint to ensure it doesn't break existing signed url logic/unit tests
fence/config-default.yaml
Outdated
@@ -487,6 +487,8 @@ AWS_CREDENTIALS: | |||
S3_BUCKETS: | |||
bucket1: | |||
cred: 'CRED1' | |||
# optionally you can manually specify an s3-compliant endpoint for this bucket | |||
endpoint_url: 'https://cleversafe.example.com/' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do you still need a bucket region
when using cleversafe?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think so
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it looks like the code is using the region, so i'm not sure how it's working in prod... maybe they configured it with a fake region field. if not, it would fail trying to get the region from aws
Co-Authored-By: Pauline Ribeyre <ribeyre@uchicago.edu>
This pull request introduces 1 alert when merging eacd20a into d3ce3bc - view on LGTM.com new alerts:
|
…dpoint for s3 is specified
New Features
Breaking Changes
Bug Fixes
Improvements
Dependency updates
Deployment changes