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

Status:Active» Fixed

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.

Title:Eroor "Undefined index: URL in ldap_sso_path_excluded_from_sso" in ldap_sso.moduleLDAP 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.

Thanks, now it's OK.

Status:Fixed» Closed (fixed)

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

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

Status:Active» Needs work

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

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.

Issue summary:View changes

A stupid mistake correction.