Skip to content

Add Linux support for update-ca-trust and update-ca-certificates (RHEL, CentOS, Fedora, Debian, Ubuntu...)#2

Merged
FiloSottile merged 2 commits intoFiloSottile:masterfrom
chlunde:update-ca-trust
Jul 4, 2018
Merged

Add Linux support for update-ca-trust and update-ca-certificates (RHEL, CentOS, Fedora, Debian, Ubuntu...)#2
FiloSottile merged 2 commits intoFiloSottile:masterfrom
chlunde:update-ca-trust

Conversation

@chlunde
Copy link
Copy Markdown
Contributor

@chlunde chlunde commented Jun 26, 2018

Hmm, are there any better tools than "tee" to install files using sudo? :)

@chlunde
Copy link
Copy Markdown
Contributor Author

chlunde commented Jun 28, 2018

Dropping this as per-user setup has been implemented

@chlunde chlunde closed this Jun 28, 2018
@FiloSottile
Copy link
Copy Markdown
Owner

Hey, I'd still like to merge this as part of the Linux support. The current code only supports Firefox, but I want it to also work with system stores. Is that ok?

@FiloSottile FiloSottile reopened this Jun 28, 2018
@adamdecaf
Copy link
Copy Markdown
Contributor

I want it to also work with system stores. Is that ok?

Same. I can help get debian flavors working.

Use update-ca-certificates on Debian/Ubuntu based systems, and
update-ca-trust when detected on RHEL/CentOS/Fedora.
@chlunde chlunde changed the title Add support for update-ca-trust (RHEL, CentOS, Fedora, ...) Add Linux support for update-ca-trust and update-ca-certificates (RHEL, CentOS, Fedora, Debian, Ubuntu...) Jun 28, 2018
@chlunde
Copy link
Copy Markdown
Contributor Author

chlunde commented Jun 28, 2018

Sure, no problem. I updated with Debian support too, tested briefly on raspbian, but I'm not sure how you would like this to work. Should it be behind a flag?

PS! The package with certutil is called nss-tools on Fedora

@adamdecaf
Copy link
Copy Markdown
Contributor

What would the flag be? Why?

@FiloSottile FiloSottile merged commit 05189bc into FiloSottile:master Jul 4, 2018
@FiloSottile
Copy link
Copy Markdown
Owner

Thank you!

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