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

johnbarclay’s picture

Status: Active » Fixed
trumanru’s picture

Priority: Major » Minor
Status: Fixed » Needs work

It's not true for now. I have errors in "Recent log messages" (admin/reports/dblog):

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).

File ldap_sso.module contains these lines:

90:  if ($path) {
91:    // don't derive
92:  }
93:  elseif ($_SERVER['URL'] == '/index.php') {
94:    $path = $_GET['q'];
95:  }
96:  else {
97:    $path = ltrim($_SERVER['URL'], '/'); // cron.php, etc.
98:  }

As you see lines numbered 93 & 97 have been trying to receive value from $_SERVER array by 'URL' index.

johnbarclay’s picture

Title: Eroor "Undefined index: URL in ldap_sso_path_excluded_from_sso" in ldap_sso.module » LDAP SSO: Error "Undefined index: URL in ldap_sso_path_excluded_from_sso" in ldap_sso.module
Status: Needs work » Fixed

This 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.

trumanru’s picture

Thanks, now it's OK.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

RoyK’s picture

Status: Closed (fixed) » Active

Looks 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

johnbarclay’s picture

Status: Active » Needs work

We need a patch that accommodates the various web server environments on this and doesn't throw errors.

johnbarclay’s picture

Status: Needs work » Fixed

#6 is old code. This particular ldap sso issue is closed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

A stupid mistake correction.