Andi Gutmans wrote: > If this is indeed a use-case for CGI then I think it's a valid argument > and we are probably better off supporting it. I did not remember shebang > works for real CGI. > There are still plenty of use-cases for running CGI (as opposed to > FastCGI) in the industry. > Question now is whether this should be done in the scanner or in the > SAPI. I'll check with Dmitry who made this change. > > Andi > Probably in sapi, so it only affects cgi and cli, and doesn't reintroduce bug 46844. OTOH, skipping shebang on included files could be a feature, if someone uses cgi and uses files both as entrypoint and included scripts. A compromise might be skipping shebang line only when it's followed by <?php, but it's hackish.
-- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php