Skip to content
This repository was archived by the owner on Jan 1, 2021. It is now read-only.

Turn on TLS and cert generation by default#563

Merged
SvenDowideit merged 1 commit intoboot2docker:masterfrom
aanand:turn-on-tls
Oct 14, 2014
Merged

Turn on TLS and cert generation by default#563
SvenDowideit merged 1 commit intoboot2docker:masterfrom
aanand:turn-on-tls

Conversation

@aanand
Copy link
Copy Markdown
Contributor

@aanand aanand commented Oct 10, 2014

@SvenDowideit
Copy link
Copy Markdown
Contributor

much yay - thought it should also be documented in this PR

@bfirsh bfirsh mentioned this pull request Oct 13, 2014
@aanand aanand force-pushed the turn-on-tls branch 2 times, most recently from 22d8c03 to 3f9a161 Compare October 13, 2014 12:44
@aanand
Copy link
Copy Markdown
Contributor Author

aanand commented Oct 13, 2014

Note added to README.

@bfirsh
Copy link
Copy Markdown
Contributor

bfirsh commented Oct 13, 2014

@aanand boot2docker also listens on port 2376 now

@aanand
Copy link
Copy Markdown
Contributor Author

aanand commented Oct 13, 2014

@bfirsh updated.

@tianon
Copy link
Copy Markdown
Contributor

tianon commented Oct 13, 2014

Fair enough; LGTM

(although worth noting that @bfirsh is trying to snake you with boot2docker/boot2docker-cli#271 changing the path 😈)

@bfirsh
Copy link
Copy Markdown
Contributor

bfirsh commented Oct 13, 2014

@aanand perhaps this should be updated to reflect boot2docker/boot2docker-cli#271

@aanand
Copy link
Copy Markdown
Contributor Author

aanand commented Oct 13, 2014

@bfirsh done.

@tianon
Copy link
Copy Markdown
Contributor

tianon commented Oct 13, 2014

LGTM

@bfirsh bfirsh added this to the 1.3.0 milestone Oct 13, 2014
@SvenDowideit
Copy link
Copy Markdown
Contributor

LGTM

@SvenDowideit
Copy link
Copy Markdown
Contributor

the PRs are merged, so merging

SvenDowideit added a commit that referenced this pull request Oct 14, 2014
Turn on TLS and cert generation by default
@SvenDowideit SvenDowideit merged commit 68be44e into boot2docker:master Oct 14, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants