Skip to content

Expand usage of LimitWikiAccess middleware #937

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

AndrewKostka
Copy link
Contributor

@AndrewKostka AndrewKostka commented May 22, 2025

Bug: T394986

@AndrewKostka AndrewKostka marked this pull request as draft May 22, 2025 08:48
@AndrewKostka AndrewKostka changed the title Refactor LimitWikiAccess middleware Expand usage of LimitWikiAccess middleware May 22, 2025
@AndrewKostka AndrewKostka marked this pull request as ready for review May 22, 2025 14:44
abort(403);
}

if (!$wikiManager->wiki) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suspect this block should be higher in the middleware; for any random non-existing wiki I think this middleware currently returns a 403 not 404

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was the existing behavior, but I agree that it makes more sense to swap it around.

@AndrewKostka AndrewKostka force-pushed the refactor-LimitWikiAccess branch from f4cb4ea to f1c4b8b Compare May 30, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants