Geal / PilotSSH-scripts

A collection of scripts usable with the PilotSSH iOS app

http://pilotssh.com

MIT License

52 stars 14 forks source link

maximum size of JSON message too low. #18

Closed zu2 closed 11 years ago

zu2 commented 11 years ago

JSON message more than 8,193 bytes causes "Parse error The script returned an empty string".　this limit be written somewhere?

So,

Geal commented 11 years ago

8k should not be a problem, normally. Is that for the process listing script? There might be a bug in there. Could you launch the script from command line, then paste the output in http://jsonlint.com/ ? It will help me fix the script.

zu2 commented 11 years ago

I am testing on Linux (CentOS5) and iPhone 5.

I test with simple script like this.

!/bin/bash

cat ~/.pilotssh/process/test.json.txt

At first, I use a non error JSON file ( 8,145 bytes). I added a blank to the file one by one. It became the error at 8,194 bytes.

Thanks.

Geal commented 11 years ago

I just tested with a 17 Kb file, and it's running fine. Could you provide me with your file, so that I can fix it?

zu2 commented 11 years ago

I want to send a file for debugging. How do I send the file?

Geal commented 11 years ago

Please send it to contact@pilotssh.com

Geal commented 11 years ago

I just released the version 1.1.0, and it seems the bug is not present in this version. Could you try to update? (This version was in review when you posted the issue).

zu2 commented 11 years ago

I just upgrade to version 1.1.0, that works fine. thanks.

Geal commented 11 years ago

I'm glad to hear it :smile:

Don't hesitate to come back to me if you have other issues!