nextcloud / nextcloud/ios

User (with space character) can not login, not authorized

Open
#1,332 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
Swift
Stars
2.5k
Forks
1k
Avg merge
2d 18h
Merged PRs (30d)
13

Description

Expected behaviour

In iOS app, logging in to a server with the username/password and should be able to see content.

Actual behaviour

Logging in when a username contains a space, able to grant access and then get a 401 when I get to the files page

When I viewed the NGINX logs, there is a plus added where the space is, causing the user name to be invalid.

Example:
Username of Test User turns into Test+User when logging in, which I assume causes the failure

This appeared to have been fixed with #493

Steps to reproduce

Create a username with a space and try to login via the iOS app

Reasoning or why should it be changed/implemented?

Unable to login via the login flow.

However, I was able to bypass this issue by logging in with a QR code

iOS version

13.5.1

App version

3.0.0.76

Server configuration

Operating system:

Debian 10

Web server:

Apache/2.4.38

Database:

Postgres 12.2

PHP version:

PHP 7.3.17

Nextcloud version: (see Nextcloud admin page)

18.0.4

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No source file or test is named. Reproduce the login flow with a username containing a space, then inspect the iOS login path and compare it with the QR-code path and the behavior described in #493. Done means the username remains valid and the files page does not return a 401.

Written by the indexing model from the issue text.

Assessment

Tech stack
ios, swift
Domain
authentication, mobile
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
28/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.