always set a user if we got asked for one

[nilsding]

No description provided.

[nilsding]

/backport to stable-3.16

[github-actions]

Artifact containing the AppImage: nextcloud-appimage-pr-7974.zip

SHA256 checksum: bc03ee8bb261fbf1a319cc2038e1ce1453e23d41422f8c4827673e7dc7a5cdb5

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[mid1221213]

Hello,

I have a problem with this PR: it breaks "basic auth" login method for the desktop client. The reason is there: the reply->abort() in this very PR makes the reply->error() become QNetworkReply::OperationCanceledError instead of QNetworkReply::AuthenticationRequiredError, thus setting _resultGet = LoginFlowV2 instead of _resultGet = Basic.

I chose not to open an issue because I'm not sure the authType Basic is supposed to be still supported, but as it is in the code I guess yes. I can open an issue if you mind.

FWIW, my use case is a "special" server setup that needs, for now, HTTP Auth method.