-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Add libnss3-tools installation to chrome and edge image #1567
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes to Nodes should be applied to this file https://github.com/SeleniumHQ/docker-selenium/blob/trunk/NodeBase/Dockerfile.txt
Corrected |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still missing docs showing how to use the feature. Could you please add them to the README?
Added usage to readme |
The point would be that usually companies allows no access to external
repos, so cannot do the install part of the libnss3. It is usually allows
to get docker images but cannot do any apt-get. Ok, frankly speaking here I
am a bit selfish, and this would help me, rather create an image at home
with libnss install then use that image to add certs within enterprise
environment.
Feel free to reject the PR if you think it would serve too narrow user base.
Attila
…On Wed, 11 May 2022 at 11:13 Diego Molina ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In README.md
<#1567 (comment)>
:
> +```bash
+FROM selenium/node-edge:latest
+
+USER root
+
+COPY certs/ /etc/certs # copy over the certificate file
+
+RUN mkdir -p -m755 /home/seluser/.pki/nssdb \ #create nssdb folder
+ && certutil -d sql:/home/seluser/.pki/nssdb -N --empty-password \ # create new db without password
+ && certutil -d sql:/home/seluser/.pki/nssdb -A -t "C,," -n companyca -i /etc/certs/companeca.pem \ #trust company CA
+ && pk12util -d sql:/home/seluser/.pki/nssdb -i client_cert.p12 -W password_of_clent_cert # client certificate install
+
+```
+This way the certificates will be installed and the node will start automatically as before.
+
+___
+
Apologies if I am being too exhaustive in the review, but if a user needs
to create their own image to install the certificates, what is the benefit
of installing the tools beforehand?
This PR could be only adding this example to the documentation, and
showing how the tools can be installed.
Am I missing something?
—
Reply to this email directly, view it on GitHub
<#1567 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADK3FFYGVHFZTDJREH4H5L3VJN23DANCNFSM5VOJ7VPA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
@algoritmus thanks for being open about it. I am not sure if it is a common use case or not. These images try to have as few things as possible to cover most use cases. However, I know some people ask every now and then about installing certificates. If you want, I'd be happy to receive a contribution to the README showing how to do it. Would you be OK with that? |
Sure, no problem.
…On Thu, 12 May 2022 at 16:36 Diego Molina ***@***.***> wrote:
@algoritmus <https://github.com/algoritmus> thanks for being open about
it. I am not sure if it is a common use case or not.
These images try to have as few things as possible to cover most use
cases. However, I know some people ask every now and then about installing
certificates. If you want, I'd be happy to receive a contribution to the
README showing how to do it. Would you be OK with that?
—
Reply to this email directly, view it on GitHub
<#1567 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADK3FF3EZBWJDIXQY4AVETDVJUJQTANCNFSM5VOJ7VPA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Thanks for contributing to the Docker-Selenium project!
A PR well described will help maintainers to quickly review and merge it
Before submitting your PR, please check our contributing guidelines, applied for this repository.
Avoid large PRs, help reviewers by making them as simple and short as possible.
Description
Added libnss3-tools package to image in order to be able to add custom certificates, WAs, etc to nssdb, or be able to create new nssdb. Nssdb certificate storage is used by chromium based browsers (Chrome, Edge)
Motivation and Context
In order to be able to add custom certificates, WAs, etc to nssdb, or be able to create new nssdb. Nssdb certificate storage is used by chromium based browsers (Chrome, Edge)
Types of changes
Checklist