POST /register downcases capitals in usernames enforces behaviour that spec defines as optional

The spec for POST /_matrix/client/v3/register states that there are two options for handling invalid usernames (like non-lowercase ones):

server MUST either map the provided username onto a user_id in a logical manner, or
reject any username which does not comply to the grammar with M_INVALID_USERNAME.

However, the test fails any servers that do the second option:

  ❌ TestRegistration/parallel/POST_/register_downcases_capitals_in_usernames (40ms)
    apidoc_register_test.go:84: MatchResponse got status 400 want 200 - http://127.0.0.1:32872/_matrix/client/v3/register => {"errcode":"M_INVALID_USERNAME","error":"Username user-UPPER contains disallowed characters or spaces: identifier contains invalid characters"}

complement/tests/csapi/apidoc_register_test.go

Lines 79 to 91 in 7b38a63

    
           // sytest: POST /register downcases capitals in usernames 
        
           t.Run("POST /register downcases capitals in usernames", func(t *testing.T) { 
        
           	t.Parallel() 
        
           	reqBody, _ := startUIASession(t, unauthedClient, "user-UPPER", "sUp3rs3kr1t", nil) 
        
           	res := unauthedClient.Do(t, "POST", []string{"_matrix", "client", "v3", "register"}, client.WithJSONBody(t, reqBody)) 
        
           	must.MatchResponse(t, res, match.HTTPResponse{ 
        
           		StatusCode: 200, 
        
           		JSON: []match.JSON{ 
        
           			match.JSONKeyTypeEqual("access_token", gjson.String), 
        
           			match.JSONKeyEqual("user_id", "@user-upper:hs1"), 
        
           		}, 
        
           	}) 
        
           })

Navigation Menu

Repository navigation

`POST /register downcases capitals in usernames` enforces behaviour that spec defines as optional #956

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

	// sytest: POST /register downcases capitals in usernames
	t.Run("POST /register downcases capitals in usernames", func(t *testing.T) {
	t.Parallel()
	reqBody, _ := startUIASession(t, unauthedClient, "user-UPPER", "sUp3rs3kr1t", nil)
	res := unauthedClient.Do(t, "POST", []string{"_matrix", "client", "v3", "register"}, client.WithJSONBody(t, reqBody))
	must.MatchResponse(t, res, match.HTTPResponse{
	StatusCode: 200,
	JSON: []match.JSON{
	match.JSONKeyTypeEqual("access_token", gjson.String),
	match.JSONKeyEqual("user_id", "@user-upper:hs1"),
	},
	})
	})

POST /register downcases capitals in usernames enforces behaviour that spec defines as optional #956

Description

Activity

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

`POST /register downcases capitals in usernames` enforces behaviour that spec defines as optional #956