I am on it. Will make the changes and update soon. I would also like to
know how you solved the warning issues my tests were giving?
Thanks


On Fri, Apr 26, 2013 at 12:19 PM, Gina Trapani <notifications@github.com>wrote:

> In webapp/_lib/class.Session.php:
>
> > @@ -71,10 +74,16 @@ public static function getLoggedInUser() {
> >       * @param Owner $owner
> >       */
> >      public static function completeLogin($owner) {
> > +		
> > +		$config = Config::getInstance();
> > +		$_SESSION[$config->getValue('source_root_path')]['session_key']=md5(microtime().$_SERVER[REMOTE_ADDR]) ;
>
> Okay. To guarantee uniqueness, let's do two things:
>
>    - Check if $_SERVER[REMOTE_ADDR] isset before referencing it
>    - Set a unique index on the key field in the session table, and on
>    insert catch a PDOException that complains if another matching session ID
>    is inserted. If so, regenerate and try again -- put this in a loop.
>
> Thank you!
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ginatrapani/ThinkUp/pull/1564/files#r3972093>
> .
>



-- 
-- 
*Hemant Kumar Singh*
III Year,
B.E.(Hons) Chemical Engineering
M.Sc. (Hons) Physics
e-mail: *Gmail <unizen01@gmail.com> , BITSmail <f2010580@bits-pilani.ac.in>*

contact: +919529068585,+917891160904

*Birla Institute of Technology & Science,* Pilani

Pilani Campus, Rajasthan 333 031 02, INDIA