Skip to content

Fix Path Style Access Setting Priority#55439

Merged
original-brownbear merged 2 commits intoelastic:masterfrom
original-brownbear:55407
Apr 20, 2020
Merged

Fix Path Style Access Setting Priority#55439
original-brownbear merged 2 commits intoelastic:masterfrom
original-brownbear:55407

Conversation

@original-brownbear
Copy link
Copy Markdown
Contributor

Fixing obvious bug in handling path style access if it's the only setting overridden by the
repository settings.

Closes #55407

Fixing obvious bug in handling path style access if it's the only setting overridden by the
repository settings.

Closes #55407
@original-brownbear original-brownbear added >bug :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v8.0.0 v7.8.0 v7.7.1 labels Apr 20, 2020
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed (:Distributed/Snapshot/Restore)

Copy link
Copy Markdown
Contributor

@ywelsch ywelsch left a comment

Choose a reason for hiding this comment

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

LGTM

@original-brownbear
Copy link
Copy Markdown
Contributor Author

Thanks Yannick!

@original-brownbear original-brownbear merged commit d960a30 into elastic:master Apr 20, 2020
@original-brownbear original-brownbear deleted the 55407 branch April 20, 2020 08:56
original-brownbear added a commit that referenced this pull request Apr 20, 2020
* Fix Path Style Access Setting Priority (#55439)

Fixing obvious bug in handling path style access if it's the only setting overridden by the
repository settings.

Closes #55407
original-brownbear added a commit that referenced this pull request Apr 20, 2020
* Fix Path Style Access Setting Priority

Fixing obvious bug in handling path style access if it's the only setting overridden by the
repository settings.

Closes #55407
@original-brownbear original-brownbear restored the 55407 branch August 6, 2020 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v7.7.1 v7.8.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

S3 Repo plugin "path_style_access" option insufficient alone

5 participants