Skip Menu |

This queue is for tickets about the Net-Google-Calendar CPAN distribution.

Report information
The Basics
Id: 69179
Status: resolved
Priority: 0/
Queue: Net-Google-Calendar

People
Owner: PLYTLE [...] cpan.org
Requestors: len.budney [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: 1.0
Fixed in: (no value)



Subject: Google's Calendar atom includes a doctype
I believe this completely breaks Net::Google::Calendar. The feed returned by Google includes "<!doctype html>", which although pointless, causes the fatal error, "StartTag: invalid element name." Net::Google::Calendar should do something defensive about that. If it can't convince XML::Atom::Thing to be tolerant of doctype statements, then Net::Google::Calendar should strip them before submitting the atom for parsing.
I'm not seeing the same thing w/ the doctype - the feeds I see are either "<?xml" or "<entry". Can you reconfirm this is still an issue, and post some code if it it? Thanks. On Wed Jun 29 09:16:47 2011, BUDNEY wrote: Show quoted text
> I believe this completely breaks Net::Google::Calendar. > > The feed returned by Google includes "<!doctype html>", which although > pointless, causes the > fatal error, "StartTag: invalid element name." > > Net::Google::Calendar should do something defensive about that. If it > can't convince > XML::Atom::Thing to be tolerant of doctype statements, then > Net::Google::Calendar should strip > them before submitting the atom for parsing.
My apologies for not attaching test code originally. I now can't find the script I was working on, and am not able to reproduce the problem. This doesn't prove it doesn't exist, of course, since I didn't save the code and don't remember what it was doing when it got the error. If you want to close the issue, pending someone giving a proper report, it's OK with me. On Mon Mar 26 11:12:33 2012, PLYTLE wrote: Show quoted text
> I'm not seeing the same thing w/ the doctype - the feeds I see are > either "<?xml" or "<entry". Can you reconfirm this is still an issue, > and post some code if it it? Thanks. > > On Wed Jun 29 09:16:47 2011, BUDNEY wrote:
> > I believe this completely breaks Net::Google::Calendar. > > > > The feed returned by Google includes "<!doctype html>", which although > > pointless, causes the > > fatal error, "StartTag: invalid element name." > > > > Net::Google::Calendar should do something defensive about that. If it > > can't convince > > XML::Atom::Thing to be tolerant of doctype statements, then > > Net::Google::Calendar should strip > > them before submitting the atom for parsing.
> >
No worries - closed. On Sat Apr 07 12:45:14 2012, BUDNEY wrote: Show quoted text
> My apologies for not attaching test code originally. I now can't find > the script I was working on, and am not able to reproduce the problem. > This doesn't prove it doesn't exist, of course, since I didn't save
the Show quoted text
> code and don't remember what it was doing when it got the error. > > If you want to close the issue, pending someone giving a proper
report, Show quoted text
> it's OK with me. > > On Mon Mar 26 11:12:33 2012, PLYTLE wrote:
> > I'm not seeing the same thing w/ the doctype - the feeds I see are > > either "<?xml" or "<entry". Can you reconfirm this is still an
issue, Show quoted text
> > and post some code if it it? Thanks. > > > > On Wed Jun 29 09:16:47 2011, BUDNEY wrote:
> > > I believe this completely breaks Net::Google::Calendar. > > > > > > The feed returned by Google includes "<!doctype html>", which
although Show quoted text
> > > pointless, causes the > > > fatal error, "StartTag: invalid element name." > > > > > > Net::Google::Calendar should do something defensive about that. If
it Show quoted text
> > > can't convince > > > XML::Atom::Thing to be tolerant of doctype statements, then > > > Net::Google::Calendar should strip > > > them before submitting the atom for parsing.
> > > >
> >