[wp-trac] [WordPress Trac] #9763: Please include IdeaWebServer in $is_apache (in wp-includes/vars.php)
WordPress Trac
wp-trac at lists.automattic.com
Thu Jul 14 11:18:59 UTC 2011
#9763: Please include IdeaWebServer in $is_apache (in wp-includes/vars.php)
-------------------------+-----------------------------
Reporter: burghardt | Owner:
Type: enhancement | Status: new
Priority: low | Milestone: Future Release
Component: General | Version: 2.7.1
Severity: normal | Resolution:
Keywords: needs-patch |
-------------------------+-----------------------------
Comment (by hakre):
Replying to [comment:2 Denis-de-Bernardy]:
> This part is not good/potentially buggy:
>
> {{{
> + if ($location[0] == '/')
> + $location = get_bloginfo('url') . $location;
> +
> }}}
>
> is it needed, or was it just something you added as an aside?
This should not be needed for the webserver technically. I smell that
apache ''might'' (would be good to know actually) modify the `Location:`
response header with relative URIs into ones with absolute URIs which is
the '''requirement''' by the HTTP standard.
So this plugin is not only fixing the issue with webserver recognition but
also with those parts of the code (those parts have been reduced over the
last months however) that do wrong redirects. It's a common mistake and I
assume that many plugins are failing here as well. I think that's the
reason why this has been added as well, but I can not suggest to take that
part in.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/9763#comment:12>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list