- Issue created by @damienmckenna
- First commit to issue fork.
- π³π±Netherlands batigolix Utrecht
batigolix β changed the visibility of the branch 3.0.x to hidden.
- π³π±Netherlands batigolix Utrecht
There were some conflicts as a result of π Replace usage of 'whitelist' with 'allowlist' Active
I fixed those so that it can be reviewed again
- π³π±Netherlands arjenk
Shield/Basic Auth trigger flood checking early in the bootstrap before the flood_control.module file is loaded. That was the reason that flood_control_get_white_list_ips() was not found. This solution now uses FloodWhiteList::getAllowedlistIps() instead of the function in the module file.
- tested without the patch: error
- tested with the patch : no problems
- the code validates in phpcs/phpstan
- pipeline is happy
- the last changes from #3534294 are merged Now that this issue is closed, please review the contribution record.
As a contributor, attribute any organization helped you, or if you volunteered your own time.
Maintainers, please credit people who helped resolve this issue.
-
batigolix β
committed 55512ffe on 3.0.x
[#3478825] feat: Incompatibility with Basic Auth / Shield? By:...
-
batigolix β
committed 55512ffe on 3.0.x