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

automod match_link fixes #432

Merged
merged 5 commits into from
Dec 29, 2023
Merged

Conversation

metal0
Copy link
Contributor

@metal0 metal0 commented Nov 26, 2023

Fixes ZDEV-64
Fixes ZDEV-66

Why is this impl so cringe?

turns out im lazy. also i didnt want to touch the regex itself because i dont want to deal with potential new bypasses originated due to changing it. and it's also tricky since a non-encoded url can probably have [](). and still be valid

@cla-bot cla-bot bot added the cla-signed label Nov 26, 2023
@metal0 metal0 requested a review from almeidx December 28, 2023 12:24
@almeidx almeidx requested a review from Dragory December 28, 2023 13:01
backend/src/utils.ts Outdated Show resolved Hide resolved
backend/src/utils.ts Outdated Show resolved Hide resolved
@metal0 metal0 requested a review from almeidx December 29, 2023 03:34
@metal0 metal0 force-pushed the matchlinks-fixes branch 2 times, most recently from 89ae47c to d4c591b Compare December 29, 2023 03:37
@almeidx almeidx merged commit 7c94694 into ZeppelinBot:master Dec 29, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants