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.

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

[bug] post counters should not include direct messages #3504

Open
codl opened this issue Nov 1, 2024 · 2 comments
Open

[bug] post counters should not include direct messages #3504

codl opened this issue Nov 1, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@codl
Copy link

codl commented Nov 1, 2024

Describe the bug with a clear and concise description of what the bug is.

Post counters on profiles and on the instance landing page count all posts, including direct messages. This seems questionable privacy-wise, you could infer how many direct messages someone is sending by monitoring how fast their post count is increasing vs how many posts you can actually see.

What's your GoToSocial Version?

v0.17.1

GoToSocial Arch

amd64 binary

What happened?

I posted a direct message and my post count went up by 1 on my public profile, so did the global count on the landing page.

What you expected to happen?

Post count does not go up.

How to reproduce it?

Post a direct message to no-one.

Anything else we need to know?

One could argue that private posts should also not be counted, or even unlisted posts. Idk about that but I am certain that direct messages should not be counted.

@codl codl added the bug Something isn't working label Nov 1, 2024
@tsmethurst
Copy link
Contributor

Ah yes we should change this indeed, thanks :)

@Booklordofthedings
Copy link

Hi, id be willing to work on a fix for that if thats fine

The fix would likely be to edit the query in internal\db\bundb\instance.go: CountInstanceStatuses, to exclude everything thats also in internal\gtsmodel\conversation.go: ConversationToStatus

Potentially there should be version of that function that does a full count, but everything that currently calls it seems to be public facing in a way where we would rather like to hide it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants