search

Cumulus / Syndic

RSS and Atom feed parsing
MIT License
34 stars 13 forks source link

Cannot parse feed #12

Closed Chris00 closed 10 years ago

Chris00 commented 10 years ago

The library cannot parse the following Atom feed. Maybe it does not respect the standard but IMHO this library should do its best to parse feeds coming from standard tools.

abbysmal commented 10 years ago

If i'm not mistaking that's what the unsafe functions are for currently: if a feed isn't fully compliant with the RFC you can still access its informations via the unsafe functions. Maybe a better solution can be found but that's the way it is currently, maybe @dinosaure have further ideas on how to handle it in the future, though.

Chris00 commented 10 years ago

If xml is the content of the above feed;

let o1 = Syndic.Atom.unsafe xml;;
Exception: Xmlm.Make(String)(Buffer).Error (_, -95440847).
Chris00 commented 10 years ago

This was because the XML feed was already parsed. The error should be more explicit.

Chris00 commented 10 years ago

@amirmc With this patch, your feed parses!