Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

synapse/app/admin_cmd.py still using removed storage.presence #10128

Closed
callahad opened this issue Jun 4, 2021 · 1 comment · Fixed by #10154
Closed

synapse/app/admin_cmd.py still using removed storage.presence #10128

callahad opened this issue Jun 4, 2021 · 1 comment · Fixed by #10154
Labels
T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.

Comments

@callahad
Copy link
Contributor

callahad commented Jun 4, 2021

synapse/app/admin_cmd.py still using removed storage.presence:
from synapse.replication.slave.storage.presence import SlavedPresenceStore

Originally posted by @vitlav in #9820 (comment)
@anoadragon453 anoadragon453 added the T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. label Jun 7, 2021
@anoadragon453
Copy link
Member

anoadragon453 commented Jun 9, 2021 •
edited
Loading

This file is used for the exfiltration of historical data in case of GDPR requests. Definitely something we'd like to have working when required, and we should probably fix it relatively soon (this broke in v1.33.0).

@anoadragon453 anoadragon453 mentioned this issue Jun 9, 2021
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Clean up a broken import in admin_cmd.py matrix-org/synapse
2 participants
@callahad @anoadragon453