-
Notifications
You must be signed in to change notification settings - Fork 350
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 TWIM-bot(?) eating links #2398
Comments
causes e.g. #2399 it might not be twim-bot, but element doing broken things... https://matrix.to/#/!kOXQjUbOfdYNCtRmZv:matrix.org/$4uFWF1B0XrW6YCmuWdGdvMoF56ITgny5NooEdBHZMio?via=matrix.org&via=element.io&via=envs.net |
Considering the nheko post was done on probably nheko (@deepbluev7 please confirm) https://github.com/haecker-felix/hebbot/blob/b230bd3749a90a9f6bb642536074154f3658a92a/src/render.rs#L347-L353 has also a bug since it missed 2 things. |
On further investigation its because the first regex checks for start of line. Thats not given due to the braces. the second one checks for a space before. Thats also not given. @haecker-felix Is the space on the second regex a typo or a bug? 🤔 I can make a PR if needed I think |
Do I read it correctly, that that regex is checking for room matrix.to links starting with a #? That only works for aliases sent by clients not following the matrix spec, since the spec suggests identifiers should be escaped, which means links start with EDIT: Seems like it works on the body of the message, which usually doesn't have a matrix.to link at all, so it probably isn't trying to match against a matrix.to link but instead against any alias. |
and the given regex can also not work with aliases in brackets e.g. |
I can't remember anymore how / why I did the regex in this way, too long ago. If it causes problems, please feel free to open a PR. |
hm but what causes the user ( |
might be limited to matrix.to links.
The text was updated successfully, but these errors were encountered: