Add support for multiple links in atom

rbren / rss-parser

A lightweight RSS parser, for Node and the browser

MIT License

1.38k stars 209 forks source link

Add support for multiple links in atom #235

Closed dmeijboom closed 2 years ago

dmeijboom commented 2 years ago

This PR contains the following features:

Add support for multiple links in atom (https://github.com/rbren/rss-parser/pull/235/commits/8df7b52a53eb157291eed242c81018e0fba0df27)
Fix null reference in rdf:about parsing (https://github.com/rbren/rss-parser/pull/235/commits/118e8624ca20a8dec2a7b71cac7a0014235c84ab)
Update most packages with known vulnerabilities (https://github.com/rbren/rss-parser/pull/235/commits/94cfb47c3afc1c49cd50b05fe0738deafb7fc5f3)

dmeijboom commented 2 years ago

~The rdf about stuff seems to be broken on master:~

  26) Parser
       should parse itunes keywords:
     Uncaught AssertionError: expected [TypeError: Cannot read properties of undefined (reading 'rdf:about')] to equal null
      at /Users/dmeijboom/DevProjects/personal/rss-parser/rss-parser/test/parser.js:23:22
      at Timeout._onTimeout (lib/utils.js:84:29)
      at listOnTimeout (node:internal/timers:559:17)
      at processTimers (node:internal/timers:502:7)

Edit: fixed!

dmeijboom commented 2 years ago

Repository doesn't seem to be maintained anymore. Merged at: https://github.com/dillendev/rss-parser