Ranchero-Software / NetNewsWire

RSS reader for macOS and iOS.
https://netnewswire.com/
MIT License
8.35k stars 530 forks source link

Link not opening when tapping on title in specific feed #3043

Open brentsimmons opened 3 years ago

brentsimmons commented 3 years ago

App Store Connect report:

“Source link not opening for articles when tap on title. Tested with: http://www.eldiadevalladolid.com/rss/DVlocal.xml

babbage commented 3 years ago

This feed isn't a valid RSS feed. Presumably most pertinently, though the validator doens't point this out, the contents of the <link> tags are wrapped in <![CDATA[ markup—which is only intended for use in <description> content, I believe? The feed also wraps title elements in the same markup.

If so, this is probably NNW displaying works-as-intended behavior, as the feed doesn't contain valid URLs?

It would be possible, of course, to strip the CDATA markup from around links... would that actually be desirable?

brentsimmons commented 3 years ago

The CDATA sections should be fine. I don’t know what the problem is here — I suspect a bug in our RSS parser.

stuartbreckenridge commented 3 years ago

There is no url being stored for each article in the database for this feed when it's a local account. (When it's coming via Feedbin it seems ok.)

Screenshot 2021-06-24 at 8 32 42 AM
vincode-io commented 3 years ago

Right. It is probably a parser bug.