View Issue Details

Jump to NotesJump to History
This issue affects 1 person(s).
 4
IDProjectCategoryView StatusDate SubmittedLast Update
10473Bug reportsRemoteControlpublic2016-02-15 23:372016-02-16 01:52
ReporterSorix Assigned Toc_schmitz  
PriorityhighSeverityblock 
Status closedResolutionwon't fix 
Product Version2.50.x 
Summary10473: JSON-RPC order requirements
Description

LimeSurvey's JSON-RPC doesn't fit with JSON specifications. To make requests for LimeSurvey via JSON-RPC I need to keep the right order of parameters and LimeSurvey just ignores keys' values.

To fit JSON specifications LimeSurvey's Remote Control need to look only at key's value and ignore order of parameters.

Steps To Reproduce

Let's send request to LimeSurvey via JSON-RPC:

{
"method": "get_session_key",
"params": {
"login": "admin",
"password": "zzz"
},
"id": 1
}

Ok, session id is received. But what if I try to change parameter's name to something random?

{
"method": "get_session_key",
"params": {
"random_text": "admin",
"hahaha": "zzz"
},
"id": 1
}

Session key is received again! But what if I change order of parameters and give right names to it? Nothing works.

TagsNo tags attached.
Bug heat4
Complete LimeSurvey version number (& build)160204
I will donate to the project if issue is resolvedNo
Browser
Database type & versionany
Server OS (if known)any
Webserver software & version (if known)any
PHP Versionany
Git branch / Pull Request URL

Users monitoring this issue

Sorix

Activities

c_schmitz

c_schmitz

2016-02-16 01:52

administrator   ~35192

Yes, order is required, but you are assuming that you need to use key->value pairs which are not needed.

$sSessionKey= $client->call('get_session_key', array('admin','password'));

is enough.

Issue History

Date Modified Username Field Change
2016-02-15 23:37 Sorix New Issue
2016-02-15 23:37 Sorix Issue Monitored: Sorix
2016-02-16 01:52 c_schmitz Note Added: 35192
2016-02-16 01:52 c_schmitz Status new => closed
2016-02-16 01:52 c_schmitz Assigned To => c_schmitz
2016-02-16 01:52 c_schmitz Resolution open => won't fix