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

Urdr bypasses 2FA upon basic auth #555

Open
jonandernovella opened this issue Aug 8, 2022 · 6 comments
Open

Urdr bypasses 2FA upon basic auth #555

jonandernovella opened this issue Aug 8, 2022 · 6 comments
Assignees
Labels
bug Something isn't working security Related to security wontfix This will not be worked on

Comments

@jonandernovella
Copy link
Contributor

jonandernovella commented Aug 8, 2022

Describe the bug

When 2FA is enabled on your Redmine account, urdr allows you to bypass this using simple username and password login.

Steps to reproduce

  1. Enable 2FA on a Redmine account
  2. Log in to urdr. No 2FA required. An api token is given to the urdr user which can be used to report time
@jonandernovella jonandernovella added bug Something isn't working security Related to security labels Aug 8, 2022
@viklund
Copy link
Member

viklund commented Aug 8, 2022

So if it's possible to login with only password using the API when 2FA is enabled in redmine then it's a redmine bug.

I don't quite remember how urdr authenticates against redmine, but it could be interesting to know if it's the same behaviour if it's a new user to urdr that have 2 factor enabled.

In any case, this has a low priority.

@jonandernovella
Copy link
Contributor Author

Related to https://www.redmine.org/issues/35001

@jonandernovella
Copy link
Contributor Author

@viklund As of Redmine version 5 (https://github.com/redmine/redmine/releases/tag/5.0.2) users with 2FA enabled on Redmine will not be able to authenticate using basic authentication with username and password.

Urdr users with active sessions will still be able to use the system but afterwards urdr will presumably not be usable by users with 2FA enabled.

@viklund
Copy link
Member

viklund commented Aug 12, 2022

I'm currently working on getting Redmine v5 working. Though there are some problems with a few plugins. I guess you'll have to disable 2fa before using urdr?

@jonandernovella
Copy link
Contributor Author

I'm currently working on getting Redmine v5 working. Though there are some problems with a few plugins. I guess you'll have to disable 2fa before using urdr?

I think I will create an issue to implement 2FA on urdr. Feel free to downprioritise it :)

@jonandernovella
Copy link
Contributor Author

#560 should address this issue

@viklund viklund added the wontfix This will not be worked on label Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working security Related to security wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants