-
Notifications
You must be signed in to change notification settings - Fork 15
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
MFA not working with APP_BASE set to something #59
Comments
Hi, i have potentially found the issue. go-passbolt uses the URL instead of the Message for MFA detection because the Message changes based on the Servers default language and used to break MFA on non english installations. The Problem is here: https://github.com/passbolt/go-passbolt/blob/e13f484bcbacb8830afe9aa7ad4fb77ee3efbaed/api/api.go#L65 For now you could temporary disable MFA on the Account you are using for go-passbolt-cli. |
I have the fix implemented and login is still work with my Passbolt instance. If you are able to compile yourself, testing the fix would be appreciated. |
Hello @speatzle, many thanks for the fix. I have no knowledge of go and I don't have a build environment ready, but I'll try to build myself in order to test. |
I confirm the fix works, many thanks |
Describe the bug:
Hello, I'm trying to access my self-hosted installation of passbolt with go-passbolt-cli and I'm always receiving the following error whatever method I set in
--mfaMode
My passbolt is started with
APP_BASE
set to/passbolt
and if I comment outAPP_BASE
the MFA works correctly.To Reproduce:
Just start passbolt with
APP_BASE
environment variable set to/passbolt
, for example with dockerOutput when using --debug (you should censor this):
If I masked too much in the below output please let me know.
Passbolt Server Version (please complete the following information):
go-passbolt-cli Version (please complete the following information):
Additional context
None
The text was updated successfully, but these errors were encountered: