Added verify option to docs#2341
Closed
NigelVanHattum wants to merge 4 commits into
Closed
Conversation
2 tasks
Comment on lines
+186
to
+188
| - `ssl_verify` - whether or not to verify SSL certificates. By default SSL certificates are verified. You can provide the following values: | ||
| False - do not validate SSL certificates. SSL will still be used (unless use_ssl is False), but SSL certificates will not be verified. | ||
| path/to/cert/bundle.pem - A filename of the CA cert bundle to use. |
Contributor
There was a problem hiding this comment.
Please wrap to 80 chars 🙏
Contributor
There was a problem hiding this comment.
And let us know when #5732 is approved in order to do a final check here 🙂
jorgeorpinel
added
the
⌛ status: wait-core-merge
Contributor
|
Thanks @NigelVanHattum ! I've merged the restyled version of this PR |
Contributor
|
Thanks guys. I didn't actually review this closely since the core PR looked like an early WIP at that point... But np, I'll make any copy edits or questions in another PR if needed 👍 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added Docs for verify option to s3 remotes: Pull request
❗ Please read the guidelines in the Contributing to the Documentation list if you make any substantial changes to the documentation or JS engine.
🐛 Please make sure to mention
Fix #issue(if applicable) in the description of the PR. This causes GitHub to close it automatically when the PR is merged.Please choose to allow us to edit your branch when creating the PR.
Thank you for the contribution - we'll try to review it as soon as possible. 🙏