When the node.get browse form is submitted, I end up getting logged out of the site, the result is 'Could not find the node.' and I get a warning 'Invalid argument supplied for foreach() in... .../services/services.module on line 435'. I am using the authkey but only sessid is enabled.

Comments

marcingy’s picture

marcingy commented
Assigned: Unassigned » marcingy

i'll take a look at this over the weekend.

marcingy’s picture

marcingy commented
Status: Active » Fixed

Couldn't replicate the being logged out but I got the invalid argument warning. We were assuming keys all the time.

greenskin’s picture

greenskin commented

Perhaps it is an issue with my setup. I shall investigate further. When you said you were assuming keys all the time, what did you mean?

My security settings:
Authorization module: key authentication
Use keys: unchecked
Token expiry time: 30
Use sessid: checked

marcingy’s picture

marcingy commented

As in there was an error in the code that meant we always tried to add an additional value to the args even when keys weren't being used.

greenskin’s picture

greenskin commented

I downloaded the latest dev release dated June 6 and the the node returns successfully with no logout and with just the sessid.

Athough, when I disable all security I get 'Authentication failed: 1'.

marcingy’s picture

marcingy commented

By disabling all security did you just uncheck keys and sessions or disable the module?

greenskin’s picture

greenskin commented

I left the keys auth module enabled but unselected 'key authentication' from the settings page so that it simply was:
Authorization module: --Select a authorization module

Which should use 'an' instead of 'a' in '--Select a authorization module', though is a separate issue.

marcingy’s picture

marcingy commented

under the current release if no authenication is enabled the module will automatically fail on authenication. If you want to run with no authenication, disable keys and sess id. There is a patch at moment http://drupal.org/node/465546 which needs review which will actually introduce an explict no authenication method.

greenskin’s picture

greenskin commented

Ok, thanks.

marcingy’s picture

marcingy commented
Status: Fixed » Closed (fixed)