Skip to content

Conversation

@S7evinK
Copy link
Collaborator

@S7evinK S7evinK commented Nov 25, 2025

This should fix #3669
We'd potentially send an empty string to GenerateCreateContent, even if we set the correct roomVersion before.

Pull Request Checklist

Signed-off-by: Till Faelligen <2353100+S7evinK@users.noreply.github.com>
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 52.21%. Comparing base (12b7038) to head (5dcf908).

Files with missing lines Patch % Lines
clientapi/routing/createroom.go 0.00% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (75.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3670      +/-   ##
==========================================
- Coverage   52.24%   52.21%   -0.04%     
==========================================
  Files         524      524              
  Lines       48399    48399              
==========================================
- Hits        25288    25273      -15     
- Misses      19602    19616      +14     
- Partials     3509     3510       +1     
Flag Coverage Δ
unittests 52.21% <0.00%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@S7evinK
Copy link
Collaborator Author

S7evinK commented Nov 25, 2025

Any feelings about bumping the default room version to 12? @kegsay @devonh

Copy link
Member

@devonh devonh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice fix! Surprised we haven't run into this before.

@devonh
Copy link
Member

devonh commented Nov 26, 2025

Any feelings about bumping the default room version to 12? @kegsay @devonh

I don't think that would be an issue. I don't have much context on how stable Dendrite is with v12. That would be my only hesitation.

@bones-was-here
Copy link

Participation in v12 rooms has been solid so far, but we haven't created many.
In a certain case gomatrixserverlib tries to parse a domain out of the room ID #3671 so probably that should be fixed before changing the default.

This patch seems to work for allowing the default to be configured to 12.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Room creation fails when default version is 12 in config

5 participants