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

SecureDrop Updater Tails Admin Password Prompt Appears to Fail When Clicking OK #3642

Closed
huertanix opened this issue Jul 14, 2018 · 13 comments · Fixed by #5169
Closed

SecureDrop Updater Tails Admin Password Prompt Appears to Fail When Clicking OK #3642

huertanix opened this issue Jul 14, 2018 · 13 comments · Fixed by #5169
Assignees

Comments

@huertanix
Copy link
Member

Description

When using the SecureDrop updater GUI tool to update SecureDrop, correctly entering the Tails admin password and clicking OK with a trackpad fails and invokes a "Administrator password is incorrect" error, while hitting the Enter key instead of clicking OK works.

Steps to Reproduce

After a successful upgrade to SecureDrop 0.7.0, the SecureDrop Updater appeared to inform me I can upgrade to 0.8.0. Once I began that upgrade, I was prompted for the Tails admin password, which I entered and used the journalist workstation's built-in trackpad to click the OK button on the prompt, after which it failed and I had to click Update Now again to restart the upgrade process, only to have it fail the same way a few times. On the final try, I used the Enter key instead of clicking OK and it accepted the admin password and completed the upgrade.

Expected Behavior

Clicking OK after entering a correct Tails admin password should accept the password and continue the upgrade.

Actual Behavior

Clicking OK after entering a correct Tails admin password fails and tells me the password is wrong.

Comments

This was on a Tails laptop that had only just been upgraded from 0.6.0 to 0.7.0 via the command line. Once the 0.7.0 upgrade was complete, the more modern SecureDrop updater prompted me to go to the shiny chrome future that is 0.8.0 and I very much wanted to.

@huertanix huertanix changed the title SecureDrop Updater Tails Admin Password Prompt Appear to Fail When Clicking OK SecureDrop Updater Tails Admin Password Prompt Appears to Fail When Clicking OK Jul 14, 2018
@redshiftzero
Copy link
Contributor

thanks for filing this @huertanix! unfortunately I was not able to reproduce this in Tails 3.8, can you confirm that was the version of Tails you used?

And can you provide the error message, i.e. what happened when it failed? If you are able to provide it, the full output of the details tab in the GUI is very helpful since that output is very verbose (please be sure to grep through the output and remove any onion addresses or HidServAuth cookies)

@kushaldas
Copy link
Contributor

I could not reproduce it for 0.7.0 on Tails 3.6.

@redshiftzero
Copy link
Contributor

closing this, but let us know if you have more information or see it again!

@huertanix
Copy link
Member Author

I ran into this again but this time I was already at 0.8.0 (just cloned the repo today but the GUI updater popped up anyway after running ./securedrop-admin tailsconfig) for a journalist (not admin) Tails drive. I had a very easy and specific Tails admin password and it would fail every time I tried it, whether with a mouse click or enter key this time. Here's the tab output:

When the update begins, this area will populate with output.
Fetching and verifying latest update... (5 mins remaining)
INFO: Applying SecureDrop updates...
INFO: Checking for SecureDrop updates...
Fetching origin
INFO: Update needed
INFO: Verifying signature on latest update...
gpg: key 0x310F561200F4AD77: public key "SecureDrop Release Signing Key" imported
gpg: no ultimately trusted keys found
gpg: Total number processed: 1
gpg: imported: 1
INFO: Signature verification successful.
Note: checking out '0.8.0'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

git checkout -b

HEAD is now at e2c2f07... SecureDrop 0.8.0
INFO: Updated to SecureDrop 0.8.0.
Checking dependencies are up to date... (2 mins remaining)
INFO: Virtualenv already exists, not creating
INFO: Checking Python dependencies for securedrop-admin
INFO: Python dependencies for securedrop-admin upgraded
INFO: Finished installing SecureDrop dependencies
Configuring local Tails environment... (1 min remaining)
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
Administrator password incorrect. Exiting upgrade - click Update Now to try again.
Fetching and verifying latest update... (5 mins remaining)
INFO: Applying SecureDrop updates...
INFO: Checking for SecureDrop updates...
Fetching origin
INFO: All updates applied
Checking dependencies are up to date... (2 mins remaining)
INFO: Virtualenv already exists, not creating
INFO: Checking Python dependencies for securedrop-admin
INFO: Python dependencies for securedrop-admin are up-to-date
INFO: Finished installing SecureDrop dependencies
Configuring local Tails environment... (1 min remaining)
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
Administrator password incorrect. Exiting upgrade - click Update Now to try again.
Fetching and verifying latest update... (5 mins remaining)
INFO: Applying SecureDrop updates...
INFO: Checking for SecureDrop updates...
Fetching origin
INFO: All updates applied
Checking dependencies are up to date... (2 mins remaining)
INFO: Virtualenv already exists, not creating
INFO: Checking Python dependencies for securedrop-admin
INFO: Python dependencies for securedrop-admin are up-to-date
INFO: Finished installing SecureDrop dependencies
Configuring local Tails environment... (1 min remaining)
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
Administrator password incorrect. Exiting upgrade - click Update Now to try again.
Fetching and verifying latest update... (5 mins remaining)
INFO: Applying SecureDrop updates...
INFO: Checking for SecureDrop updates...
Fetching origin
INFO: All updates applied
Checking dependencies are up to date... (2 mins remaining)
INFO: Virtualenv already exists, not creating
INFO: Checking Python dependencies for securedrop-admin
INFO: Python dependencies for securedrop-admin are up-to-date
INFO: Finished installing SecureDrop dependencies
Configuring local Tails environment... (1 min remaining)
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
Administrator password incorrect. Exiting upgrade - click Update Now to try again.
Fetching and verifying latest update... (5 mins remaining)
INFO: Applying SecureDrop updates...
INFO: Checking for SecureDrop updates...
Fetching origin
INFO: All updates applied
Checking dependencies are up to date... (2 mins remaining)
INFO: Virtualenv already exists, not creating
INFO: Checking Python dependencies for securedrop-admin
INFO: Python dependencies for securedrop-admin are up-to-date
INFO: Finished installing SecureDrop dependencies
Configuring local Tails environment... (1 min remaining)
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
Administrator password incorrect. Exiting upgrade - click Update Now to try again.
Fetching and verifying latest update... (5 mins remaining)
INFO: Applying SecureDrop updates...
INFO: Checking for SecureDrop updates...
Fetching origin
INFO: All updates applied
Checking dependencies are up to date... (2 mins remaining)
INFO: Virtualenv already exists, not creating
INFO: Checking Python dependencies for securedrop-admin
INFO: Python dependencies for securedrop-admin are up-to-date
INFO: Finished installing SecureDrop dependencies
Configuring local Tails environment... (1 min remaining)
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
INFO: Configuring Tails workstation environment
INFO: You'll be prompted for the temporary Tails admin password, which was set on Tails login screen
Administrator password incorrect. Exiting upgrade - click Update Now to try again.

@kushaldas
Copy link
Contributor

I still can not reproduce it :( I installed fresh Tails 3.8, and there I moved to develop and then did tailsconfig first. I moved to 0.7.0 and then disconnected and connected back the network, that fired up the updater, and typed the sudo password manually. If I am on 0.8.0, I could not manage to get the application up automatically.

@conorsch
Copy link
Contributor

Reopening for further investigation based on @huertanix 's report.

@conorsch conorsch reopened this Aug 28, 2018
@eloquence
Copy link
Member

We should see if we can get a repro or root cause on this ahead of the next release. Making a note to set aside some time for additional investigation in the next sprint.

@eloquence eloquence added this to the 0.10.0 milestone Sep 7, 2018
@redshiftzero redshiftzero modified the milestones: 0.10.0, 0.11.0 Oct 25, 2018
@eloquence
Copy link
Member

eloquence commented Oct 31, 2018

Sadly, we're having to bump this again. On the upside, we've not seen any reports in support about this issue, but we should still make one more attempt to repro before the next release.

@zenmonkeykstop
Copy link
Contributor

Cannot reproduce on current Tails (3.10.1).

Downgraded to 0.7.0, bumped the network connection, the updater appears.
Clicked Update Now, the admin password comes up.
Put in admin password, clicked OK. Update completes without error to 0.9.1.
Bumped network again, updater appears, repeat update process, clicking OK instead of hitting Return. Update completes without error to 0.10.0

Repeated the above 3 times. No errors.

@eloquence
Copy link
Member

Given multiple unsuccessful repro attempts, no other reports, and no obvious potential causes in the code itself, we'll have to close this for now. @huertanix, thanks for reporting -- next time you encounter this issue, if you can get a clean repro, let's sync up in real-time to see if we can get to the bottom of this.

@conorsch
Copy link
Contributor

During QA for 1.2.1 today I encountered this issue: despite my very carefully typing in the Tails admin password, the graphical updater told me it was incorrect. Confirmed I was using the correct pw by running sudo echo hello in the terminal, no problems.

@eloquence eloquence removed this from the 0.11.0 milestone Feb 19, 2020
@eloquence eloquence reopened this Feb 19, 2020
@eloquence
Copy link
Member

Got a report of this from a production user so bumping in priority.

@eloquence
Copy link
Member

Saw this myself today with first attempt failing (though password was correct), second attempt working. Poked a bit at the Qt input but isolating that I don't see any issues at least in my Ubuntu dev env -- e.g., it ignores control characters and only outputs what I input. My hypothesis, FWIW, is that the logic for setting the password via a child process sometimes doesn't work (some timing issue).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants