sizzlelab / side

t3rc
6 stars 3 forks source link

Fix new api #45

Closed mataanin closed 12 years ago

mataanin commented 12 years ago

a simple html file with following code, doesn't work:

<html>
<body>
<form action="http://jimu.cs.hut.fi/SIDE_REST_API/session" method="post">
<input type="text" name="username" value="maksim@androidaalto.org" /> <br />
<input type="text" name="password" value="111111" /> <br />
<input type="text" name="projectCode" value="100036" /> <br />
<input type="text" name="mobileID" value="Nokia" /> <br />
<input type="submit" name="submit" value="submit" /> <br />
</form>
</body>
</html>

Server returns the following:

<br />
<b>Notice</b>:  Undefined variable: sessionID in         <b>/var/www/html/SIDE_REST_API/SESSION_CONTROLLER.php</b> on line <b>181</b><br />
"{\"sessionID\":\"1Ix9Ly5i9Jd29jOK379Sm6Lp\"}"
mataanin commented 12 years ago

now it returns. Still working bad: "{\"sessionID\":\"TxpYK0SshMG20fj6fJ05CNnA\"}"

zxhwd2008 commented 12 years ago

Don't know where is the bad thing? it returns the sessionID, and you could get it now

mataanin commented 12 years ago

This is not valid JSON. Java standard parser classes can't parse it.

On Thu, Nov 17, 2011 at 10:00 PM, Xiaohang < reply@reply.github.com

wrote:

Don't know where is the bad thing? it returns the sessionID, and you could get it now


Reply to this email directly or view it on GitHub: https://github.com/sizzlelab/side/issues/45#issuecomment-2781059

mataanin commented 12 years ago

Any update on this one?

When should we expect it fixed?

On Thu, Nov 17, 2011 at 10:07 PM, Maksim Golivkin maksim@golivkin.euwrote:

This is not valid JSON. Java standard parser classes can't parse it.

On Thu, Nov 17, 2011 at 10:00 PM, Xiaohang < reply@reply.github.com

wrote:

Don't know where is the bad thing? it returns the sessionID, and you could get it now


Reply to this email directly or view it on GitHub: https://github.com/sizzlelab/side/issues/45#issuecomment-2781059

zxhwd2008 commented 12 years ago

Done

mataanin commented 12 years ago

Fixed.

Does new api implement new error code functionality?

mataanin commented 12 years ago

Postponing due to low priority.