distributed auth patch [#9177]

distributed auth no longer works on the first time because user_load() doesn't return empty for user_load(array('name' => "$name@$server")) ... it returns the anonymous user object.

small patch to fix this... making things like drupal.module auth work again.

Comment	File	Size	Author
#3	dist-auth-patch.diff	1.57 KB	walkah
#3
	user-fix-dist-auth.diff	1.13 KB	walkah

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Junyor CreditAttribution: Junyor commented 9 July 2004 at 18:43

Thanks, that solved the problem I was having. Seems to work well in limited testing.

Comment #2

Dries CreditAttribution: Dries commented 10 July 2004 at 07:10

I wonder when/how this broke? Strange fix.

Why is the following check _inside_ the loop:

if (variable_get('user_register', 1) == 1

Comment #3

walkah CreditAttribution: walkah commented 15 July 2004 at 14:55

File	Size
dist-auth-patch.diff	1.57 KB

judging from the fact that the object returned from user_load() consists of only the roles array (with 'anonymous user'), i'm guessing it broke around the introduction of multiple roles/user. i.e. the anonymous user is no longer an empty object.

attached is a new patch that moves the user_load inside the other check (good idea).