The Wayback Machine - https://web.archive.org/web/20220916071923/https://github.com/FiloSottile/mkcert/pull/100
Skip to content
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

Jump to bottom

nss: write a notice about being asked for db password #100

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

nss: write a notice about being asked for db password #100

wants to merge 1 commit into from
+18 −2

Conversation

adamdecaf
Copy link
Contributor

@adamdecaf adamdecaf commented Jan 7, 2019

Issue: #50
@adamdecaf adamdecaf mentioned this pull request Jan 7, 2019
Open
@FiloSottile
Copy link
Owner

FiloSottile commented Jan 8, 2019

But most users don't have a master password, so it would be confusing wouldn't it?

Can we instead print something like (hint, this is your Firefox master password) if certutil is taking more than 1s?
@adamdecaf
Copy link
Contributor Author

adamdecaf commented Jan 9, 2019 •
edited

Good point. I'll try and do that.
FiloSottile
FiloSottile requested changes Feb 2, 2019
View changes
Copy link
Owner

@FiloSottile FiloSottile left a comment

#100 (comment)
@adamdecaf adamdecaf force-pushed the notice-for-ff-master-password branch from 58e37de to 4ba70fe Compare Feb 7, 2019
@adamdecaf
Copy link
Contributor Author

adamdecaf commented Feb 7, 2019 •
edited

This works locally for me, I tried with and without a firefox master password.

$ go run . -install 
Using the local CA at "/Users/adam/Library/Application Support/mkcert" ✨
The local CA is now installed in the system trust store! ⚡️
Enter Password or Pin for "NSS Certificate DB":
(hint, this is your Firefox master password))
 
The local CA is now installed in the Firefox trust store (requires browser restart)! 🦊
The local CA is now installed in Java's trust store! ☕️

$ go run . -install 
Using the local CA at "/Users/adam/Library/Application Support/mkcert" ✨
The local CA is now installed in the system trust store! ⚡️
The local CA is now installed in the Firefox trust store (requires browser restart)! 🦊
The local CA is now installed in Java's trust store! ☕️
@lopezator
Copy link

lopezator commented Apr 14, 2020

Is this going to be merged anytime? I've been also hit by this.
@algorun77
Copy link

algorun77 commented Apr 19, 2021

Hello, same here, is it planned to be merged someday ? It would really be helpful to have that hint because right now, the password asked is confusing
Aurelius333
Aurelius333 reviewed Aug 9, 2021
View changes
truststore_nss.go
case <-errChan:
return // next profile
case <-timeout:
log.Print("\n(hint, this is your Firefox master password))")
Copy link

@Aurelius333 Aurelius333 Aug 9, 2021

Choose a reason for hiding this comment

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

Firefox is now calling it a primary password rather than a master password: https://support.mozilla.org/en-US/kb/primary-password-replacing-master-password.
@jesusdf
Copy link

jesusdf commented Mar 23, 2022

2022 and we still have this issue ongoing.

What about "Enter your Firefox primary password (or leave blank if no password is set)"?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
6 participants
@adamdecaf @FiloSottile @lopezator @algorun77 @jesusdf @Aurelius333