Felipe Zipitría
Felipe Zipitría
That way we can just include it in the corresponding regex using the regexp-assemble `include` directive.
Awesome. I think we should take a note in the data file where the wrappers come from (https://www.php.net/manual/en/wrappers.php), and [add it to the issue](https://github.com/coreruleset/coreruleset/issues/2621) that tracks the list of files...
Added to the list that tracks updates. Closing.
Now that we merged the 934120 update, you can use the same technique used there in lines https://github.com/coreruleset/coreruleset/blob/51b5d465d9b7db3ec7654762165eafaa6c9b79ae/util/regexp-assemble/data/934120.data#L14-L19 to include the url-schemes list.
@karelorigin Just a question: are you going to have time to update this one, or do you want me to add the latest changes? I know you are very busy...
Hi @karelorigin! Sorry for the delay. You can of course refer to it from any file. The include is "static", as it will pick files only from the `util/regexp-assemble/data/include/` directory,...
@karelorigin Hey! Any news on this one? Do you need help?
Solved conflicts.
Fixed also: - moved to including url-schemes - fixed `file://` inclusion as url-scheme 🤦 - fixed dependent rule 934120.
Thanks @karelorigin for pushing this change!