Skip to content
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

[wip] Add --insecure flag to skip certificate validation #1814

Merged
merged 1 commit into from
Sep 2, 2016

Conversation

vadmeste
Copy link
Member

@vadmeste vadmeste commented Sep 1, 2016

Remarks:
*) I upgraded to session v8 because we have the new insecure flag
*) I print certificate information when --debug is enabled.. otherwise I guess it will be just annoying for the user to always see it

@@ -47,6 +47,10 @@ var globalFlags = []cli.Flag{
Name: "debug",
Usage: "Enable debugging output.",
},
cli.BoolFlag{
Name: "insecure",
Usage: "Skip SSl certificate verification.",
Copy link
Member

Choose a reason for hiding this comment

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

's/SSl/SSL/g'

@codecov-io
Copy link

codecov-io commented Sep 1, 2016

Current coverage is 8.94% (diff: 13.95%)

Merging #1814 into master will decrease coverage by 0.08%

@@            master     #1814   diff @@
========================================
  Files           77        77          
  Lines         5817      5903    +86   
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
+ Hits           525       528     +3   
- Misses        5162      5244    +82   
- Partials       130       131     +1   

Powered by Codecov. Last update 530a34a...6d05544

@harshavardhana harshavardhana merged commit fd34958 into minio:master Sep 2, 2016
@vadmeste vadmeste deleted the add_insecure_flag branch May 29, 2018 20:33
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.

3 participants