[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