Skip to content

mk-ca-bundle: use secure connection to hg.mozilla.org - #1012

Closed
indygreg wants to merge 1 commit into
curl:masterfrom
indygreg:https-in-mk-ca-bundle
Closed

indygreg wants to merge 1 commit into
curl:masterfrom
indygreg:https-in-mk-ca-bundle

Conversation

@indygreg

Copy link
Copy Markdown

Previously, connections to hg.mozilla.org were insecure and could be
subject to a MitM attack. With this change, someone has to MitM you
with a certificate signed by a trusted CA, which is a bit harder.

Previously, connections to hg.mozilla.org were insecure and could be
subject to a MitM attack. With this change, someone has to MitM you
with a certificate signed by a trusted CA, which is a bit harder.
@mention-bot

Copy link
Copy Markdown

@indygreg, thanks for your PR! By analyzing the annotation information on this pull request, we identified @bagder, @gknauf and @watson81 to be potential reviewers

@bagder

bagder commented Sep 14, 2016

Copy link
Copy Markdown
Member

It would only use HTTP instead of HTTPS if you don't have curl installed or your curl has a problem to download the cert data using HTTPS. Maybe we should instead only try plain HTTP if being allowed to with a dedicated command line option?

@jay

jay commented Sep 15, 2016

Copy link
Copy Markdown
Member

It would only use HTTP instead of HTTPS if you don't have curl installed or your curl has a problem to download the cert data using HTTPS. Maybe we should instead only try plain HTTP if being allowed to with a dedicated command line option?

Definitely, see master...jay:mk-ca-bundle_https-only_by_default

@jay jay added the TLS label Sep 15, 2016
@bagder

bagder commented Sep 15, 2016

Copy link
Copy Markdown
Member

@jay : a big 👍 on that!

@bagder bagder left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'd like to see @jay's approach either replace this or getting merged into this version.

@bagder bagder closed this in 1ad2bdc Oct 24, 2016
@bagder

bagder commented Oct 24, 2016

Copy link
Copy Markdown
Member

I decided to go ahead and merge @jay's patch here since I think it was a good approach to this and makes mk-ca-bundle.pl better! Thanks both of you!

@jay

jay commented Oct 24, 2016

Copy link
Copy Markdown
Member

I forgot about this. I actually worked on the vbs too, I'll find it tomorrow (or today depending on where you're at) and merge it in.

@bagder

bagder commented Oct 24, 2016

Copy link
Copy Markdown
Member

Lovely. I've been pondering about removing the vbs version before as it hasn't gotten the same attention as the perl version, but if you're up to bringing it back to life I'm fine with that too!

jay added a commit that referenced this pull request Oct 30, 2016
- Change initial message box to mention delay when downloading/parsing.

Since there is no progress meter it was somewhat unexpected that after
choosing a filename nothing appears to happen, when actually the cert
data is in the process of being downloaded and parsed.

- Warn if OpenSSL is not present.

- Use a UTF-8 stream to make the ca-bundle data.

- Save the UTF-8 ca-bundle stream as binary so that no BOM is added.

---

This is a follow-up to d2c6d15 which switched mk-ca-bundle.vbs output to
ANSI due to corrupt UTF-8 output, now fixed.

This change completes making the default certificate bundle output of
mk-ca-bundle.vbs as close as possible to that of mk-ca-bundle.pl, which
should make it easier to review any difference between their output.

Ref: #1012
@indygreg
indygreg deleted the https-in-mk-ca-bundle branch January 27, 2017 23:43
@indygreg

Copy link
Copy Markdown
Author

I just wanted to give a heads up that http://hg.mozilla.org/ will HTTP 301 to https://hg.mozilla.org/ starting around 2017-02-01 0800 PST. I know mk-ca-bundle.pl is used in random places throughout the Internets. So if you hear about it breaking on Wednesday, that's probably why.

https://groups.google.com/d/msg/mozilla.dev.version-control/7A6WwhraKm4/QxRs2dw5BAAJ for the full announcement.

@bagder

bagder commented Jan 27, 2017

Copy link
Copy Markdown
Member

Ah right, thanks @indygreg. This will make the mk-ca-bundle.pl start to fail unconditionally unless curl is installed and working on the machine running the script. But I consider the switch a good thing nonetheless and such failures will help users to fix their stuff and get secure transfers going.

bagder added a commit that referenced this pull request Aug 18, 2017
Ref #1012
Figured-out-by: Tatsuhiro Tsujikawa
@lock lock Bot locked as resolved and limited conversation to collaborators Jan 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Development

Successfully merging this pull request may close these issues.

4 participants