Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions server/mergin/auth/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -212,12 +212,12 @@ def generate_username(cls, email: str) -> Optional[str]:
text(
"""
SELECT
replace(lower(username), :username, '0')::int AS suffix
replace(lower(username), :username, '0')::bigint AS suffix
FROM "user"
WHERE
lower(username) = :username OR
lower(username) SIMILAR TO :username_like
ORDER BY replace(lower(username), :username, '0')::int DESC
ORDER BY replace(lower(username), :username, '0')::bigint DESC
LIMIT 1;
"""
),
Expand Down
4 changes: 4 additions & 0 deletions server/mergin/tests/test_auth.py
Original file line number Diff line number Diff line change
Expand Up @@ -866,6 +866,10 @@ def test_username_generation(client):
user = add_user("testuser1")
assert User.generate_username("Testuser@example.com") == "testuser2"

# test username with crazy long int suffix
user = add_user("testuser13120931904")
assert User.generate_username("Testuser@example.com") == "testuser13120931905"


def test_server_usage(client):
"""Test server usage endpoint"""
Expand Down
Loading