gofeed icon indicating copy to clipboard operation
gofeed copied to clipboard

accept empty xml space for content:encoded

Open JLugagne opened this issue 2 years ago • 2 comments

in some cases, the space is empty to use the tag namespace as default. To manage this I have added an OR conditions in rss/parser.go.

Alternative: improve goxpp to use tag namespace as xmlns default if empty ?

JLugagne avatar Jul 12 '23 08:07 JLugagne

@JLugagne Can you show a quick example of this? I think I understand, but want to be completely sure.

It sounds like this might be better to be fixed in goxpp, but I'd like to make sure I grok what the issue is.

mmcdole avatar Jul 20 '23 04:07 mmcdole

Sure, one of the RSS feed I had to use was having xmlns:content in the root tag but in the content:enconded tag it was defining only the xmlns attribute (you can check the unit test I have added). In that case I guess that it means xmlns refer to the namespace of the tag?

JLugagne avatar Jul 20 '23 06:07 JLugagne

This should be fixed with #223, so closing. Thank you @JLugagne !

mmcdole avatar Mar 01 '24 03:03 mmcdole