1219056 - /api/2/user/<username> doesn't work if the user name contains a dot(.)

Status: RESOLVED FIXED

(support.mozilla.org :: Users and Groups, task)

Description

[Cor vL (cor-el)]

/api/2/user/<username> doesn't work if the user name contains a dot(.)
Other cases (e.g. involved=<username>) I tried work, so possibly only user is affected.
Noticed it with this user: /questions/1085818
Created a test account (cor-el.test) and that fails as well.
Works OK with the normal forum link for a user profile (/user/<username>).

Comment 1

[Michael Cooper [:mythmon]]

When you say "doesn't work", what do you see? I'm getting a 404 when I test it here. Is that right?

Comment 2

[Cor vL (cor-el)]

Yes, that is right.
If you open the page via the location bar then you get a 404 error.
Via XHR I see a "NOT FOUND" response.
[HTTP/1.1 404 NOT FOUND 585ms]

I don't know what other delimiter characters are allowed in a user name apart from a hyphen (-) and an underscore (_).

Comment 3

[Cor vL (cor-el)]

This work:
https://support.mozilla.org/api/2/question/?format=json&product=firefox&locale=en-US&ordering=-id&involved=charles.scripter

Comment 4

[Safwan Rahman (:safwan)]

Confirming as The API is also not working for me
https://support.mozilla.org/api/2/user/Safwan.rahman returning a 404 Error

Comment 5

[Safwan Rahman (:safwan)]

As I suspected, its a issue with the Django 3.x Upgrade, but actually its not.
I have tested with pre update and it seems that the issue is also present there

Comment 6

[Kadir Topal [:atopal]]

Mike, Brittany, is this something we need to fix before moving to the new AAQ flow?

Comment 7

[Safwan Rahman (:safwan)]

(In reply to Kadir Topal [:atopal] from comment #6)
> Mike, Brittany, is this something we need to fix before moving to the new
> AAQ flow?

As its not possible to get information of user having username with dot by using the API, The AAQ flow can not verify if the user is logged in. So the user who has username with dot may can not post any question. So this need to be fixed before moving to the AAQ flow

Comment 8

[Rehan Dalal (Inactive)]

Pretty easy to fix this. 1pt assigned.

Comment 9

[Michael Cooper [:mythmon]]

PR: https://github.com/mozilla/kitsune/pull/2760/
Merged and deployed.