Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Lines 93 and 97 contains a reference to an array element $_SERVER['URL']. It's wrong cause we have no an element named as 'URL'. Right choice is: $_SERVER['REQUEST_URI'].
Because of his bug we have errors:
•Notice: Undefined index: URL in ldap_sso_path_excluded_from_sso() (line 93 of C:\www\portal\sites\all\modules\ldap\ldap_sso\ldap_sso.module).
•Notice: Undefined index: URL in ldap_sso_path_excluded_from_sso() (line 97 of C:\www\portal\sites\all\modules\ldap\ldap_sso\ldap_sso.module).
Comments
Comment #1
johnbarclay CreditAttribution: johnbarclay commentedThis is fixed in 7.x-2.x-dev. See #1787816: LDAP SSO: add option to exclude some URLs (like cron.php) from automatic SSO.
Comment #2
trumanru CreditAttribution: trumanru commentedIt's not true for now. I have errors in "Recent log messages" (admin/reports/dblog):
File ldap_sso.module contains these lines:
As you see lines numbered 93 & 97 have been trying to receive value from $_SERVER array by 'URL' index.
Comment #3
johnbarclay CreditAttribution: johnbarclay commentedThis was committed on jan 7th. See http://drupalcode.org/project/ldap.git/blame/refs/heads/7.x-2.x:/ldap_ss...
-dev just means the most current commit on the branch. You have to get a new one to have the current -dev anytime a commit has been made.
Comment #4
trumanru CreditAttribution: trumanru commentedThanks, now it's OK.
Comment #6
RoyK CreditAttribution: RoyK commentedLooks like there's a bug in there after all. in ldap_sso/ldap_sso.module, lines 93 and 97, $_SERVER['URL'] used. This is not a valid $_SERVER index, but changing it to $_SERVER['PHP_SELF'] works. This is from ldap-7.x-2.0-beta3.tar.gz
roy
Comment #7
johnbarclay CreditAttribution: johnbarclay commentedWe need a patch that accommodates the various web server environments on this and doesn't throw errors.
Comment #8
johnbarclay CreditAttribution: johnbarclay commented#6 is old code. This particular ldap sso issue is closed.
Comment #9.0
(not verified) CreditAttribution: commentedA stupid mistake correction.