Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Auth::check() not working with 4.1 and debug bar #73

Closed
glendaviesnz opened this issue Jan 13, 2014 · 7 comments
Closed

Auth::check() not working with 4.1 and debug bar #73

glendaviesnz opened this issue Jan 13, 2014 · 7 comments

Comments

@glendaviesnz
Copy link

Hi

I am having problems with Auth::check() not working with Laravel 4.1 when debug bar is enabled - seems to be an issue with sessions not persisting - eg. Auth::check will work if called on same page as Auth::login($user), but on redirect to a new page it fails as the session appears to be empty.

It was running fine on 4.0, and just upgraded to 4.1 today.

I am running dev-master ,and have tried completely removing and adding again the debug bar, but the session issues persist until I comment out 'Barryvdh\Debugbar\ServiceProvider', from app.php.

Any ideas on how best to 'debug' the debug bar so I can work out what is causing this?

Thanks in advance - great tool, so keen to be able to get it working again!

Glen

@barryvdh
Copy link
Owner

Does the issue also occur on the latest release? (Require 1.* instead of dev-master)

@glendaviesnz
Copy link
Author

Beautiful - changing to 1.* fixed it - thanks - let me know if you want me to revert to dev-master if you want to run any checks to see what was causing it, otherwise I will just leave it on 1.* and you can close this.

Thanks for quick response.

@barryvdh
Copy link
Owner

Can you try removing this line? https://github.com/barryvdh/laravel-debugbar/blob/master/src/Barryvdh/Debugbar/SymfonyHttpDriver.php#L37
That is the only line I changed about the session I think.
Are you using the default session driver?

barryvdh added a commit that referenced this issue Jan 13, 2014
Could be issue for #72 and #73?
@barryvdh
Copy link
Owner

Does the above commit help?

@ghost
Copy link

ghost commented Jan 13, 2014

Had same problem at night. And looks like latest master commit fixes this issue.

@glendaviesnz
Copy link
Author

Sorry about the delay in replying - down in NZ, so was off to bed just after posting first message. Many thanks again for your quick response to this. I have reverted back to dev-master and the issue is resolved so your commit worked.
Thanks
Glen

@barryvdh
Copy link
Owner

Okay, good. Pushed a new tag.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants