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

Arisa sometimes fails to thumbnail large images #818

Open
Dhranios opened this issue Nov 22, 2023 · 1 comment
Open

Arisa sometimes fails to thumbnail large images #818

Dhranios opened this issue Nov 22, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@Dhranios
Copy link

Dhranios commented Nov 22, 2023

The Bug

Arisa is intended to add |thumbnail to large image files, however sometimes this is skipped.

Example Ticket/comments

https://bugs.mojang.com/browse/MC-266941 (it did the last one, not the first 2
https://bugs.mojang.com/browse/MC-252942?focusedId=1290184&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-1290184 (edited by mod afterward, also now hidden/deleted(I took the link from a past request of mine))
https://bugs.mojang.com/browse/MC-189332?focusedId=1268825&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-1268825 (edited by mod afterward)

Expected behavior

Make all images above a certain size thumbnails.

@Dhranios Dhranios added the bug Something isn't working label Nov 22, 2023
@Marcono1234
Copy link
Contributor

Thanks for reporting this!

For MC-266941 the reason is probably that the image reference is enclosed by *: *!2023-11-22_14.08.48.png!*
Currently the Arisa module is a bit strict when trying to detect image references, to reduce false positives.

However, I assume the logic could be refactored to first look at all the attachments of the issue, and then search for references in the form !name!, which would be less strict and probably more accurate.
But maybe the attachment name should probably still be validated to avoid malicious users uploading images with sentences or similar as name, causing Arisa to corrupt the description of issues (?).

For the comments you mentioned I unfortunately don't know what was causing this.

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

2 participants