search

eve-seat / seat

SeAT 0x. [UNSUPPORTED]
https://github.com/eveseat/seat
MIT License
69 stars 37 forks source link

Error in DashboardController.php #304

Closed ghost closed 9 years ago

ghost commented 9 years ago

Running latest dev @ 8a741c2 there is an issue with DashboardController.php@Line 68 causing the Server throw an stacktrace into its logs and browser shows 'Bad Gateway' as response.

It may is caused by empty(Session::get('valid_keys')) - if I remove that empty all is working fine, except search for non-Superuser's.

Here is a Stacktrace http://pastebin.com/VGkySVx1

eve-seat commented 9 years ago

Could you test with cd3cd11?

ccatlett1984 commented 9 years ago

tested on my install, user without any api keys. I get the warning message as expected.

ghost commented 9 years ago

The error is gone now, but it is showing a warning that I got no api keys.

If I append error message in line 70 with . count(Session::get('valid_keys')) it prints 7 as count in the Error Message - and that user has 7 keys.

eve-seat commented 9 years ago

So much derp. Apologies, please retry after 516d92c.

ccatlett1984 commented 9 years ago

now working with non-super user with keys ;)

ghost commented 9 years ago

Can confirm it as working :+1:

eve-seat commented 9 years ago

Great. I'll have to .3 to fix the shit I broke :P