Disable character range check for user IDs

[neilalexander]

This check is not safe in existing room versions because Synapse hasn't enforced it properly either.

Related: matrix-org/matrix-spec#1506, matrix-org/matrix-spec-proposals#2828

Signed-off-by: Neil Alexander git@neilalexander.dev

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 54.11%. Comparing base (eb14008) to head (a076af0).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #410      +/-   ##
==========================================
+ Coverage   54.07%   54.11%   +0.03%     
==========================================
  Files          52       52              
  Lines        5993     5993              
==========================================
+ Hits         3241     3243       +2     
+ Misses       2342     2341       -1     
+ Partials      410      409       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[neilalexander]

cc @S7evinK @devonh

[kegsay]

This is the hill I'm dying on.

matrix-org/matrix-spec#1506 (comment)

If the above clarification PR lands, then I'll admit defeat and merge this.

[bones-was-here]

Bump: that did get merged :/

[S7evinK]

Given the spec clarification.. merged.. sadly..

// Edit: Please don't hunt me down for this @kegsay