View Issue Details

IDProjectCategoryView StatusLast Update
5446Composrcore_cnspublic2023-11-18 01:49
ReporterPDStig Assigned ToPDStig  
PrioritynormalSeveritymajor 
Status resolvedResolutionfixed 
Product Version10.0.43 
Fixed in Version10.0.44 
Summary5446: Private topics get leaked when set to receive notifications for all forum topic activity
DescriptionPrivate topics will get leaked to members who have their notifications set to receive notifications for all forum topic activity.

This includes the title and URL of the PT. It also includes system messages, such as when someone is invited to the topic. It does not include the first post in the PT. I am not sure yet if it includes any further posts by members.

This is a high priority bug as it is a privacy risk.
TagsType: Legal compliance / Privacy
Attach Tags
Attached Files
Time estimation (hours)
Sponsorship open

Sponsor

Date Added Member Amount Sponsored

Activities

PDStig

2023-11-15 07:25

administrator   ~8046

I have not yet confirmed if the bug also exists in v11.

admin

2023-11-18 01:32

administrator   ~8048

Automated response: Leaked PTs to members monitoring all topic activity

Private topics will get leaked to members who have their notifications set to receive notifications for all forum topic activity.

This happens when someone is invited to the private topic. The system message triggers a standard non-PT topic notification to members who had enabled notifications for all topic activity. This is because the function for determining whether a topic is private would not work it out (set it straight to false) if the post (in this case the system message saying someone was invited to the PT) was not the topic starter.

This fix explicitly defines the system message being posted as being posted in a private topic so that only members involved in the PT get the notification.

An additional check was added to ensure dispatch_notification never passes in "null" to topic notifications for to_member_ids (which would then cause everyone to receive the notification) when sending a notification for a private topic. Instead, it will be an empty array if anything other than an array of members.

admin

2023-11-18 01:32

administrator   ~8049

Fixed in git commit 9302069df0 (https://gitlab.com/composr-foundation/composr/commit/9302069df0 - link will become active once code pushed to GitLab)

A hotfix (a TAR of files to upload) has been uploaded to this issue. These files are made to the latest intra-version state (i.e. may roll in earlier fixes too if made to the same files) - so only upload files newer than what you have already. If there are files in a hot-fix that you don't have then they probably relate to addons that you don't have installed and should be skipped. Always take backups of files you are replacing or keep a copy of the manual installer for your version, and only apply fixes you need. These hotfixes are not necessarily reliable or well supported. Not sure how to extract TAR files to your Windows computer? Try 7-zip (http://www.7-zip.org/).

PDStig

2023-11-18 01:49

administrator   ~8050

Bug also existed in, and was fixed in, v11

Issue History

Date Modified Username Field Change
2023-11-15 07:24 PDStig New Issue
2023-11-15 07:24 PDStig Status Not Assigned => Assigned
2023-11-15 07:24 PDStig Assigned To => user4172
2023-11-15 07:24 PDStig Tag Attached: Type: Legal compliance / Privacy
2023-11-15 07:25 PDStig Note Added: 0008046
2023-11-18 01:49 PDStig Note Added: 0008050