- Issue created by @nicoc
- Status changed to Postponed: needs info
about 2 years ago 5:14pm 27 February 2023 This is just a proposition, cause i saw this solution many times with this kind of errors with PHP8.1
$path = str_replace($this->decodedChars[0], $this->decodedChars[1], rawurlencode($path ?? ''));
in fact, many lines of this have to be patched for php 8.1 .
all these lines are not compatible with PHP8.1 :
L301 if (strpos($path, '//') === 0) { L306 $path = str_replace($this->decodedChars[0], $this->decodedChars[1], rawurlencode($path)); L309 if (strpos($path, '/.') !== FALSE) { L3016 if ('/..' === substr($path, -3)) { and more...
The best way for me is to test $path before you use it.
- First commit to issue fork.
- Status changed to Closed: outdated
5 days ago 3:47pm 10 May 2025 - πΊπΈUnited States smustgrave
Without steps to reproduce think this can be closed out. We run all MRs against 8.1 and 8.3, maybe even 8.4 now and if something was fully broken think we would find it. So would need steps for what could trigger this if there is an edge case. If steps can be provide feel free to re-open.
Thanks all!