cadorn / firephp-libs

FirePHP Server Libraries
http://www.firephp.org/
23 stars 15 forks source link

Undefined index: authkeys in /FirePhp/lib/Insight/Helper.php:142 #9

Open mbideau opened 13 years ago

mbideau commented 13 years ago

I get the error on a fresh install.

The bug is simple to fix : in the file /FirePhp/lib/Insight/Helper.php in the function init() replace the line : if($clientInfo = self::$instance->getClientInfo()) { by the following : if($clientInfo = self::$instance->getClientInfo() && isset($clientInfo['authkeys'])) {

Keep up the good work. Cheers from France ;-)

mbideau commented 13 years ago

sorry I've forgotten to added parenthesis like this : if(($clientInfo = self::$instance->getClientInfo()) && isset($clientInfo['authkeys'])) {

cadorn commented 13 years ago

Do you have your authkey set to *?

mbideau commented 13 years ago

Hi,

Yes I've my authkey settted.

Do you want me to send you my credentials.json and package.json files (with keys hidden) ?

I've analysed your code and the bug comes from the return values of the function getClientInfo (from my memory) ?

The correction I've submitted works well for me.


2011/5/12 cadorn < reply@reply.github.com>

Do you have your authkey set to *?

Reply to this email directly or view it on GitHub: https://github.com/cadorn/firephp-libs/issues/9#comment_1148634

cadorn commented 13 years ago

I am not convinced this key should ever not be set.

Yes, please send me your credentials.json and package.json files and modify the key instead of hiding them.

I also need the following dumps from within getClientInfo:

var_dump(Insight_Util::getallheaders()) var_dump(Insight_Util::getRequestHeader("x-wf-protocol-1")) var_dump($this->getAnnounceReceiver()->getAuthkeys()) var_dump($this->getAnnounceReceiver()->getReceivers()) var_dump($this->getUserAgent()) var_dump(Insight_Util::getRequestHeader("X-FirePHP-Version"))

As well as the request headers.

Thanks!

mbideau commented 13 years ago

Ok. I'll send you that as soon as I can (I'm a little busy these days, maybe the next week).


2011/5/16 cadorn < reply@reply.github.com>

I am not convinced this key should ever not be set.

Yes, please send me your credentials.json and package.json files and modify the key instead of hiding them.

I also need the following dumps from within getClientInfo:

var_dump(Insight_Util::getallheaders()) var_dump(Insight_Util::getRequestHeader("x-wf-protocol-1")) var_dump($this->getAnnounceReceiver()->getAuthkeys()) var_dump($this->getAnnounceReceiver()->getReceivers()) var_dump($this->getUserAgent()) var_dump(Insight_Util::getRequestHeader("X-FirePHP-Version"))

As well as the request headers.

Thanks!

Reply to this email directly or view it on GitHub: https://github.com/cadorn/firephp-libs/issues/9#comment_1175429

cadorn commented 13 years ago

Is this still relevant?

mbideau commented 13 years ago

Hi,

I'm sorry, I've forgotten this thread. I've not updated since, I will do it the next week (I'm on vacation now). I'll tell you if it is still relevant.

Thank you for asking.

Keep up the great work.


2011/8/18 cadorn < reply@reply.github.com>

Is this still relevant?

Reply to this email directly or view it on GitHub: https://github.com/cadorn/firephp-libs/issues/9#issuecomment-1841011