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

🐛 Fix spoiler and message deletion #259

Merged
merged 1 commit into from
Dec 27, 2023
Merged

🐛 Fix spoiler and message deletion #259

merged 1 commit into from
Dec 27, 2023

Conversation

ascpial
Copy link
Contributor

@ascpial ascpial commented Dec 27, 2023

Le spoiler est correctement affiché quand un fichier est transmis par le wormhole.

Maintenant, la comparaison du contenu des messages est exacte puisque l'édition des messages est supporté.
De plus, le bot vérifiera les attachment pour supprimer le bon message. Pour ce faire, il vérifie le nom du fichier, sa taille, et si il est activé en tant que spoiler ou non.

@Aeris1One
Copy link
Collaborator

Est-ce qu'il ne faudrait pas ajouter une vérification sur l'auteur ?
Un exploit simple à réaliser est de copier un message récent, de l'envoyer puis le supprimer ou le modifier, cela supprimera/modifiera le message original.

Ça peut être un peu compliqué à gérer avec les format de noms personnalisés cela dit.

@ascpial
Copy link
Contributor Author

ascpial commented Dec 27, 2023

Oui en effet, avec les noms customiser, on ne peut pas faire beaucoup plus de vérification.

@Aeris1One Aeris1One merged commit 9423065 into beta Dec 27, 2023
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wormhole remove spoilers on images Plugins/Wormhole problems related to non-mapping of messages IDs
2 participants