[development] sid not available in anonimous user

Moshe Weitzman weitzman at tejasa.com
Thu Jan 17 04:10:36 UTC 2008


i can't really get excited one way or another. if you want a universal
way to get session id, call session_id(). your patch is a very slight
improvement, i suppose.

On 1/16/08, Ivan Sergio Borgonovo <mail at webthatworks.it> wrote:
> On Wed, 16 Jan 2008 15:04:03 -0500
> "Moshe Weitzman" <weitzman at tejasa.com> wrote:
>
> > > If people find it a good idea to be able to get $user->sid even
> > > for not authenticated users... I don't mind about the
> > > implementation ;) If people think it is a bad idea, I'd like to
> > > know why.
>
> > i can't make out what your proposal is. please post a patch.
>
> Index: session.inc
> ===================================================================
> --- session.inc (revisione 419)
> +++ session.inc (copia locale)
> @@ -48,6 +48,7 @@
>    else  {
>      $session = isset($user->session) ? $user->session : '';
>      $user = drupal_anonymous_user($session);
> +    $user->sid=session_id();
>    }
>
>    return $user->session;
>
> --
> Ivan Sergio Borgonovo
> http://www.webthatworks.it
>
>


More information about the development mailing list