-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Add case for Ubuntu 16.04 #19390
Add case for Ubuntu 16.04 #19390
Conversation
Cool, thanks for testing. Only question I guess is if multiple of these are found, which should take priority? |
Are there distros with multiple path, and distinct certificate files? That seems like a difficult decision to make, if there are. But probably for now we can ignore it. |
All distros I've seen where multiple of our paths match are symlinks to one concrete file. I've yet to see a system that has multiple certificate stores with different contents in them, and if one does exist, I'm not sure we can possibly know which is the best, unless we wanted to somehow test that the certificates work. (I don't think we want to do that) |
Is this good to merge then? |
We should verify that in an actual example, are these symlinks to the same file when multiple of them exist? If they're symlinks, does this copy them correctly (should it be dereferencing the symlink)? |
I just tested this on Bash on Ubuntu on Windows (or whatever the name is), and this fixes the bug there as well. This is using Ubuntu 14.04. |
Bump. |
The only distribution I've been able to find that has both of these facts true is CentOS, which has Both of those certificate stores are symlinks to a third file, located at |
I've also seen Ubuntu installs that had multiple of these names present. Should look into whether they're all equivalent. |
I looked around in docker installs of 14.04 and 16.04 and a live install of
14.04 and couldn't find any duplicate names, so I'm not sure what
conditions need to be met for us to see that.
On Tue, Nov 29, 2016, 03:02 Tony Kelman ***@***.***> wrote:
I've also seen Ubuntu installs that had multiple of these names present.
Should look into whether they're all equivalent.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#19390 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAH_aGo47nnrUNRElz2LjXNK8X7smdHJks5rDAYxgaJpZM4K6NOC>
.
--
…-E
|
Thanks @TotalVerb for the contribution and @davidanthoff for the testing on Windows! |
This fixes #18693 for me.
cc @tkelman