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

Don't add group chats that we are not a member of #163

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Thaodan
Copy link

@Thaodan Thaodan commented Jun 23, 2022

Resolves #150 #157

@Thaodan Thaodan force-pushed the thaodan/only_add_groupchat_member branch from 9be3844 to ca1ff97 Compare June 23, 2022 17:21
@Thaodan Thaodan changed the title #150 Don't add group chats that we are not a member of Jun 23, 2022
Td also fetches updates from groups that we are not a member of
like those appear linked_chat_id.
Checking if we are a member of those groups first helps to prevent
those linked chats to pop up.

Read td's docs here for more info:
https://core.telegram.org/tdlib/docs/classtd_1_1td__api_1_1supergroup_full_info.html#aff3925ba397314a7645686c223495d09

Signed-off-by: Björn Bidar <bjorn.bidar@thaodan.de>
@Thaodan Thaodan force-pushed the thaodan/only_add_groupchat_member branch from ca1ff97 to 7406eed Compare June 23, 2022 17:34
@BenWiederhake
Copy link
Contributor

Hey there, I've collected all open PRs into my master branch. I don't make any promises about keeping it up-to-date though.

https://github.com/BenWiederhake/tdlib-purple/#new-libpurple-plugin-for-telegram

@bmomjian
Copy link

bmomjian commented Aug 29, 2023

I needed the fix for #157 due to recent malicious spamming to my account. I was able to build/install @BenWiederhake's fork on Ubuntu 20.04 by also installing gperf so configure succeeded. I assume this will fix the spamming problem, and if not I will report back.

It would be nice if a new release can be made so manual builds can be avoided. @BenWiederhake, thanks for creating this fork in the mean time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

many messages from groups that I'm not a member
3 participants