Closed (outdated)
Project:
Drupal core
Version:
7.x-dev
Component:
install system
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Nov 2010 at 20:16 UTC
Updated:
17 Mar 2023 at 12:52 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
okokokok commentedThe notice disappeared after adding
to my nginx configuration (see e.g. http://forum.nginx.org/read.php?2,3457).
But I didn't get the behavior I've learnt to expect from D6 - I'm not logged in after the setup and drush (3.3) doesn't work either.
So I'll reinstall.
Comment #2
okokokok commentedAfter reinstalling everything works as expected.
Comment #3
amateescu commentedSo we should close this?
Comment #4
okokokok commentedNot having SERVER_SOFTWARE set is not a problem with D6.
And I don't think it's part of the design that Drupal fails when SERVER_SOFTWARE is not set.
So I think it should still be open but with low priority.
Comment #5
arhak commented@#4 if you still have this issue, then don't let it close yet
@#3 an "undefined index" PHP Notice "works as designed"?
I hope not
Comment #6
amateescu commentedarhak, nope, but I thought 'After reinstalling everything works as expected.' was.
Comment #7
okokokok commented@#4 if you still have this issue, then don't let it close yet
I don't have the issue because I changed my nginx setup.
I don't know if there are many nginx setups that don't set the SERVER_SOFTWARE variable. It had just worked fine for me for over 2 years without it.
Comment #8
jeffsheltren commentedI just came across a similar issue and thought I could fix this while I was at it. In my case, OG_Mailinglist is doing it's own drupal bootstrap as part of postfix mail delivery (piping mail into Drupal), and I received a PHP notice:
Notice: Undefined index: SERVER_SOFTWARE in require_once() (line 13 of /var/www/support.tag1.it/htdocs/includes/mail.inc).
Seems like a simple isset() should work in both cases. I wasn't entirely sure if we want to set an error/warning if the variable isn't set in the install case, so for now I'm just setting it to an empty string if the variable is not set.
Attaching a patch to fix both of these notices.
Comment #9
poker10 commentedPatch does not apply. It seems like that the first change was done in #2356055: Notice in includes/mail.inc define