-
Notifications
You must be signed in to change notification settings - Fork 669
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
Check basic auth credentials in wizard #9697
Conversation
@@ -189,6 +187,11 @@ void SetupWizardController::nextStep(std::optional<PageIndex> currentPage, std:: | |||
}); | |||
|
|||
connect(messageBox, &QMessageBox::accepted, this, [this, showFirstPage]() { | |||
// when moving back to this page (or retrying a failed credentials check), we need to make sure existing cookies |
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.
Is this triggered every time we go back or only when we displayed an error?
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.
Every time the page is instantiated.
This way, users don't unexpectedly have to re-login after the wizard completes if they entered the wrong credentials.
5613bea
to
1c765a3
Compare
Kudos, SonarCloud Quality Gate passed! |
This way, users don't unexpectedly have to re-login after the wizard completes if they entered the wrong credentials.
Fixes #9623.
Please note that there is one issue which I haven't been able to fully resolve. When moving back from the last page to the second one, the wizard re-checks the auth type (which should not be necessary but also shouldn't hurt). For some reason, this time, the lookup fails. You can move forward from the first page just fine, though, and the lookup works again.