ClutchAndroid Bug.

74 views
Skip to first unread message

NasK

unread,
Oct 17, 2012, 2:12:18 AM10/17/12
to clut...@googlegroups.com
I think I found a bug in the ClutchAndroid code that needs a serious look at!!

The "doInBackground" method in the ClutchAPIClient -> AsyncTask, since my RPC port is having an issue for whatever reason, the method throws an exception which then causes the return value to be NULL.

The right return value would be something like this:                 return new StatusCodeAndResponse(500,null);

Since the code within "onPostExecute" does not check for a NULL resp, it causes the whole Android APP to crash.

What this implies is that if the live Clutch RPC is taken offline, it will take all the Android Apps with it..

Nas.

Eric Florenzano

unread,
Oct 18, 2012, 8:40:59 PM10/18/12
to clut...@googlegroups.com
Whoa, that's definitely a big issue.  I rewrote this class right before open sourcing it in order to reduce the number of dependencies, so I probably introduced this bug.  Do you mind opening an issue on the GitHub project?  If you have a patch for it, that would help a ton.  Otherwise I can get to this in the next few days.

Thanks,
Eric
Reply all
Reply to author
Forward
0 new messages