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

Re-add WikiForum extension #480

Open
yaronkoren opened this issue Jan 6, 2025 · 4 comments
Open

Re-add WikiForum extension #480

yaronkoren opened this issue Jan 6, 2025 · 4 comments

Comments

@yaronkoren
Copy link
Member

WikiForum was in Canasta at the beginning, but was then removed about a year go (via #355 and #357) because it had gone unmaintained and had several security risks. However, just today this extension was restored back to "Stable" on mediawiki.org, with the comment that "all known issues are addressed". So, should we re-add it?

@labster
Copy link

labster commented Jan 8, 2025

It may be stable but we already found another XSS bug in it. WikiForum has some deep problems.

https://issue-tracker.miraheze.org/T13064#262072

@yaronkoren
Copy link
Member Author

@labster - good to know. Do the WikiForum people know about this?

@yaronkoren
Copy link
Member Author

@labster - can you offer any more information? I don't have a Miraheze account, so I can't see that issue you're pointing to.

@labster
Copy link

labster commented Feb 10, 2025

The details in the task were:

XSS:
http://mw142.icecone.internal/w/index.php?title=Special:WikiForum&wfaction=editcategory&category=1&uselang=x-xss
2025-01-07_20-12.png (1×1 px, 61 KB)

Sources:

https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/WikiForum/+/e0bd65ef5a40526e3d8eeb2f1363e87c080a08dc/includes/WFCategory.php#502
https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/WikiForum/+/e0bd65ef5a40526e3d8eeb2f1363e87c080a08dc/includes/WFCategory.php#513
https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/WikiForum/+/e0bd65ef5a40526e3d8eeb2f1363e87c080a08dc/includes/WFCategory.php#523

Proper fix: Escape the parameters in the function WikiForumGUI::showTopLevelForm(): https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/extensions/WikiForum/+/e0bd65ef5a40526e3d8eeb2f1363e87c080a08dc/includes/WikiForumGui.php#289

Maybe @BlankEclair can give more details

Actually I've been working on a major refactor, trying to do nice things like Html::element and ResourceLoader, but I haven't even been able to get my first commit merged, so it might end up becoming a fork.

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

No branches or pull requests

2 participants