Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix bug with creating public rooms on workers #17177

Merged
merged 3 commits into from
May 13, 2024
Merged

Fix bug with creating public rooms on workers #17177

merged 3 commits into from
May 13, 2024

Conversation

erikjohnston
Copy link
Member

@erikjohnston erikjohnston commented May 13, 2024 •
edited
Loading

If room publication is disabled then creating public rooms on workers would not work.

Introduced in #16811.

Stack trace:

AttributeError: 'GenericWorkerStore' object has no attribute 'set_room_is_public'
  File "synapse/http/server.py", line 332, in _async_render_wrapper
    callback_return = await self._async_render(request)
  File "synapse/http/server.py", line 544, in _async_render
    callback_return = await raw_callback_return
  File "synapse/rest/client/room.py", line 171, in on_POST
    return await self._do(request, requester)
  File "synapse/rest/client/room.py", line 176, in _do
    room_id, _, _ = await self._room_creation_handler.create_room(
  File "synapse/handlers/room.py", line 921, in create_room
    await self.store.set_room_is_public(room_id, False)

@erikjohnston
Fix bug with creating public rooms on workers
4a45bdb 
If room publication is disabled then creating public rooms on workers
would not work.

Introduced in #16811.
@erikjohnston
Newsfile
a479298
@erikjohnston
Lint
966b404
@erikjohnston erikjohnston marked this pull request as ready for review May 13, 2024 09:34
@erikjohnston erikjohnston requested a review from a team as a code owner May 13, 2024 09:34
anoadragon453
anoadragon453 approved these changes May 13, 2024 •
edited
Loading
View reviewed changes
@erikjohnston erikjohnston merged commit a2e6f43 into develop May 13, 2024
38 checks passed
@erikjohnston erikjohnston deleted the erikj/fix_set_room branch May 13, 2024 11:12
erikjohnston added a commit that referenced this pull request May 13, 2024
@erikjohnston
Actually fix public rooms
cb81186 
See #17177.
@erikjohnston erikjohnston mentioned this pull request May 13, 2024
Merged
erikjohnston added a commit that referenced this pull request May 13, 2024
@erikjohnston
Actually fix public rooms (#17184)
59ac541 
See #17177.

I'm an idiot and moved them to the wrong store 🤦
yingziwu added a commit to yingziwu/synapse that referenced this pull request Jun 7, 2024
@yingziwu
Merge tag 'v1.108.0' into develop
bff6843 
No significant changes since 1.108.0rc1.

- Add a feature that allows clients to query the configured federation whitelist. Disabled by default. ([\#16848](element-hq/synapse#16848), [\#17199](element-hq/synapse#17199))
- Add the ability to allow numeric user IDs with a specific prefix when in the CAS flow. Contributed by Aurélien Grimpard. ([\#17098](element-hq/synapse#17098))

- Fix bug where push rules would be empty in `/sync` for some accounts. Introduced in v1.93.0. ([\#17142](element-hq/synapse#17142))
- Add support for optional whitespace around the Federation API's `Authorization` header's parameter commas. ([\#17145](element-hq/synapse#17145))
- Fix bug where disabling room publication prevented public rooms being created on workers. ([\#17177](element-hq/synapse#17177), [\#17184](element-hq/synapse#17184))

- Document [`/v1/make_knock`](https://spec.matrix.org/v1.10/server-server-api/#get_matrixfederationv1make_knockroomiduserid) and [`/v1/send_knock/`](https://spec.matrix.org/v1.10/server-server-api/#put_matrixfederationv1send_knockroomideventid) federation endpoints as worker-compatible. ([\#17058](element-hq/synapse#17058))
- Update User Admin API with note about prefixing OIDC external_id providers. ([\#17139](element-hq/synapse#17139))
- Clarify the state of the created room when using the `autocreate_auto_join_room_preset` config option. ([\#17150](element-hq/synapse#17150))
- Update the Admin FAQ with the current libjemalloc version for latest Debian stable. Additionally update the name of the "push_rules" stream in the Workers documentation. ([\#17171](element-hq/synapse#17171))

- Add note to reflect that [MSC3886](matrix-org/matrix-spec-proposals#3886) is closed but will remain supported for some time. ([\#17151](element-hq/synapse#17151))
- Update dependency PyO3 to 0.21. ([\#17162](element-hq/synapse#17162))
- Fixes linter errors found in PR #17147. ([\#17166](element-hq/synapse#17166))
- Bump black from 24.2.0 to 24.4.2. ([\#17170](element-hq/synapse#17170))
- Cache literal sync filter validation for performance. ([\#17186](element-hq/synapse#17186))
- Improve performance by fixing a reactor pause. ([\#17192](element-hq/synapse#17192))
- Route `/make_knock` and `/send_knock` federation APIs to the federation reader worker in Complement test runs. ([\#17195](element-hq/synapse#17195))
- Prepare sync handler to be able to return different sync responses (`SyncVersion`). ([\#17200](element-hq/synapse#17200))
- Organize the sync cache key parameter outside of the sync config (separate concerns). ([\#17201](element-hq/synapse#17201))
- Refactor `SyncResultBuilder` assembly to its own function. ([\#17202](element-hq/synapse#17202))
- Rename to be obvious: `joined_rooms` -> `joined_room_ids`. ([\#17203](element-hq/synapse#17203), [\#17208](element-hq/synapse#17208))
- Add a short pause when rate-limiting a request. ([\#17210](element-hq/synapse#17210))

* Bump cryptography from 42.0.5 to 42.0.7. ([\#17180](element-hq/synapse#17180))
* Bump gitpython from 3.1.41 to 3.1.43. ([\#17181](element-hq/synapse#17181))
* Bump immutabledict from 4.1.0 to 4.2.0. ([\#17179](element-hq/synapse#17179))
* Bump sentry-sdk from 1.40.3 to 2.1.1. ([\#17178](element-hq/synapse#17178))
* Bump serde from 1.0.200 to 1.0.201. ([\#17183](element-hq/synapse#17183))
* Bump serde_json from 1.0.116 to 1.0.117. ([\#17182](element-hq/synapse#17182))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anoadragon453 anoadragon453 anoadragon453 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@erikjohnston @anoadragon453