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

[BUG]2FA problem: Does not take authorization in the first go #4508

Open
hman2 opened this issue Nov 4, 2024 · 8 comments
Open

[BUG]2FA problem: Does not take authorization in the first go #4508

hman2 opened this issue Nov 4, 2024 · 8 comments
Labels
Milestone

Comments

@hman2
Copy link

hman2 commented Nov 4, 2024

Actual behaviour

-When you authorize using 2FA (here: via Free OTP) you have to perform the authentication procedure twice.

Expected behaviour

-The app should accept the authorization the first time it is presented

Steps to reproduce

  1. Start OwnCloud app and connect to the cloud
  2. You get redirected due to authorization expiration, so you enter your credentials in the Web browser that is opened for you, then you create the token for 2FA (here: via Free OTP) and enter it. In the browser the cloud opens up and also a popup comes that asks you "open in app?"
  3. Agree to opening in the app, OwnCloud app comes up again, but: complains about expiration once more, so go back to step 2 and repeat everything. This time the autorization gets accepted.

Can this problem be reproduced with the official owncloud server?
(url: https://ocis.ocis.master.owncloud.works, user: einstein, password: relativity)
Haven't tried so far.

Environment data

Android version:
10 (AOSP, Fairphone Open OS 23.02.0-rel.0 (latest version)
Device model:
Fairphone 2
Stock or customized system:
AOSP, but officially supported the manufacturer of the phone (Fairphone).
ownCloud app version:
4.4.1 (latest) - but this behaviour was also present on the two previous versions
ownCloud server version:
unknown (hosted by my employer).

Logs

Web server error log

Insert your webserver log here

ownCloud log (data/owncloud.log)

Insert your ownCloud log here
@hman2
Copy link
Author

hman2 commented Nov 4, 2024

Additional remark: This behaviour is only exhibited if the authorization gets refreshed. When you delete the account and set it up from scratch, the app will take the authorization in the first attempt!

@jesmrec
Copy link
Collaborator

jesmrec commented Nov 5, 2024

So, you have 2FA with an external OTP. The process finishes, but, you see an "expiration" message on the bottom of the screen that prompts you to a second authentication.

Maybe this is related with #4332

My guess is that the granted token is not correctly used in the first requests under a race condition

It'd help to know which kind of server and version are you using. If you type in your web browser the server URL following with /status.php, you will see some information that you can prompt to us.

For example: https://thisismyserver.com/status.php

Thanks a lot for reporting!

@hman2
Copy link
Author

hman2 commented Nov 6, 2024

{
"installed": true,
"maintenance": false,
"needsDbUpgrade": false,
"version": "",
"versionstring": "",
"edition": "",
"productname": "",
"product": ""
}

@jesmrec
Copy link
Collaborator

jesmrec commented Nov 6, 2024

@hman2 are all those fields (product, productname...) empty or any problem with copy/paste?, these are the key fields :(

@hman2
Copy link
Author

hman2 commented Nov 7, 2024

Info from our company admins: Server version is 10.15. The output I quoted is what the browser (MS Edge via internal network connection) presented to me.

@jesmrec
Copy link
Collaborator

jesmrec commented Nov 7, 2024

Ok, so it's an oC10, thanks. We'll review. Also, if you can attach logs (you'll find in settings -> logs), it could help to clarify.

@hman2
Copy link
Author

hman2 commented Nov 7, 2024

Hm, there is no settings -> logs. There is a protocol feature, is that what you mean? It's not under More or Advanced... Or are you talking about the backend? I don't have access, all I can do is ask our admins...

@jesmrec
Copy link
Collaborator

jesmrec commented Nov 7, 2024

In the mobile app, by clicking on the top left icon (hamburger icon), the side menu is open. Then, select Settings, then Logging and Log Files . Please, mark the Log HTTP requests option for a better logging of the situation.

@joragua joragua added this to the Backlog milestone Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants