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.

Sign up for GitHub

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

Jump to bottom

401 responses are ignored #334

Closed
MorrisJobke opened this issue Aug 7, 2017 · 2 comments
Closed

401 responses are ignored #334

MorrisJobke opened this issue Aug 7, 2017 · 2 comments
Labels
3. to review bug
Milestone
v2.17.6

Comments

@MorrisJobke
Copy link
Member

  • setup an account in the iOS app
  • change the password on the server
  • actual: the app retries all requests (PROPFINDS, SEARCH, GET, MKCOL, ...) that cause a 401
  • expected: the app should stop doing requests to not run into the brute force protection and ask the user for the password

See nextcloud/server#6010

We got complains that the log is spammed by to many tried logins.

cc @marinofaggiana
@MorrisJobke MorrisJobke added the bug label Aug 7, 2017
@marinofaggiana marinofaggiana added this to the v2.17.6 milestone Aug 7, 2017
marinofaggiana added a commit that referenced this issue Aug 8, 2017
@marinofaggiana
move openLoginView on AppDelegate #334
906e03e
marinofaggiana added a commit that referenced this issue Aug 8, 2017
@marinofaggiana
Move openLoginWeb for kOCErrorServerUnauthorized on networking #334
5590e5c
@marinofaggiana
Copy link
Member

@MorrisJobke fix build 00006

@MorrisJobke
Copy link
Member Author

Works 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
3. to review bug
Projects
None yet
Milestone
v2.17.6
Development

No branches or pull requests
2 participants
@MorrisJobke @marinofaggiana