Skip to content

Update README.md #186

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

Update README.md #186

wants to merge 2 commits into from

Conversation

@bt
Copy link
Contributor

@bt bt commented Jun 29, 2016

Added line to add for insecure HTTPS connections.

bt added 2 commits June 28, 2016 01:46
@bt
Update README.md
045cb64 
Added line to add for insecure HTTPS connections.
@bt
Merge pull request #1 from bt/bt-https-readme
6caed1e 
Update README.md
@bt bt mentioned this pull request Jun 29, 2016
Closed
@kadamwhite kadamwhite closed this in 978cf0b Jul 4, 2016
@kadamwhite
Copy link
Collaborator

kadamwhite commented Jul 4, 2016

Thank you! I rebased this into 978cf0b and merged it in -- I moved the warning text down to the same part of the README where authentication and security are discussed, but left things otherwise unchanged. Thank you very much for the contribution, @bt!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bt @kadamwhite