synapse icon indicating copy to clipboard operation
synapse copied to clipboard

Local media retention doesn't delete from DB

Open erikjohnston opened this issue 1 year ago • 2 comments

For media_retention.local_media_lifetime we call:

https://github.com/element-hq/synapse/blob/696cc9e802f63ba8657856d85f6982f49de14f27/synapse/media/media_repository.py#L1254

which is wrong

erikjohnston avatar Mar 08 '24 15:03 erikjohnston

Local media IDs will deleted:

https://github.com/element-hq/synapse/blob/696cc9e802f63ba8657856d85f6982f49de14f27/synapse/media/media_repository.py#L1257

https://github.com/element-hq/synapse/blob/696cc9e802f63ba8657856d85f6982f49de14f27/synapse/storage/databases/main/media_repository.py#L907-L912

But it's really not ideal.

dklimpel avatar Mar 11 '24 12:03 dklimpel

Oooh, good spot!

erikjohnston avatar Mar 11 '24 15:03 erikjohnston