Intermittent fault when importing ics file

[ghost]

When running: 'gcalcli import CalendarEvent.ics' i get an error
some of the time (maybe 1/3 or 1/5 or 1/10 of the time), and the events are not imported. The other times i get no error, and the events are imported.

Version: gcalcli v2.2 (Eric Davis) - from debian testing
The CalendarEvent.ics file: https://gist.github.com/4279685

The error:

/x/d> gcalcli import CalendarEvent.ics
Traceback (most recent call last):
File "/usr/bin/gcalcli", line 1579, in
BowChickaWowWow()
File "/usr/bin/gcalcli", line 1568, in BowChickaWowWow
gcal.ImportICS(verbose, args[0])
File "/usr/bin/gcalcli", line 1188, in ImportICS
PrintErrMsg("Error: " + e["reason"] + "\n")
TypeError: sequence index must be integer, not 'str'

[tresni]

e["reason"] is no longer valid.. I've found using e[0] gives a text representation of the error at least. e[0]["reason"] works sometimes, but I haven't had e[0] fail yet.

[ghost]

I also get an intermitten fault when running (looks like it happens at the same rate as the ics import problem, the other times it has no error, and the even is recorded in google calendar):

~> gcalcli quick '01.10.13 10:00 test'
Traceback (most recent call last):
File "/usr/bin/gcalcli", line 1579, in
BowChickaWowWow()
File "/usr/bin/gcalcli", line 1546, in BowChickaWowWow
gcal.QuickAdd(unicode(args[1], locale.getpreferredencoding()))
File "/usr/bin/gcalcli", line 1025, in QuickAdd
PrintErrMsg("Error: " + e["reason"] + "\n")
TypeError: sequence index must be integer, not 'str'

[tresni]

Check out my fork at https://github.com/tresni/gcalcli . If nothing else it will get you corrected error messages so you can have some idea what's going on.

[ghost]

Thanks! Now i get: https://dl.dropbox.com/u/9974176/error_gcalcli.txt

[insanum]

I've also seen the "Moved Temporarily" fault from Google's servers as well. It's random and rare. Haven't seen it in a while. As you know, all you have to do is try again and it will succeed. Annoying. We could add exception handling to automatically retry but this is just a simple script and I'd rather just focus on features and keep it simple with proper error messages when things go wrong.

I've merged in tresni's fix for the exception handling problem.