Latest Update Breaks Safari about rss-parser HOT 4 CLOSED

Thanks! Fix is published as 2.10.5.

Out of curiosity, how are you using this in-browser? dist/rss-parser.js should have const removed after going through webpack.

from rss-parser.

Also, if you have any ideas on how best to test for this, let me know.

from rss-parser.

You are right! I'm using it in Meteor, in a method. Not sure if you know Meteor, but methods are made available to both client and server (so that the client can do optimistic updates). Things were working out of the box, so I didn't write any code to specifically limit the function to the server side. But then this latest update obviously caused a problem, as the client-side evaluation on safari was complaining.

What I'd normally do is wrap code like this in a server-only block if the client complains. But in 2.10.2 (where I started), things were working so I didn't think a wrap was necessary.

Let me know if you have any questions -- I think this is probably an edge case, as typically the client wouldn't be evaluating server-side code.

from rss-parser.

Ahh that makes sense. I'd been considering rewriting index.js with ES6 features since dist/ files will be translated using babel, but it's good to know that it'd break some workflows. If I do go that route I'll release it as a major version so it won't break Meteor builds.

Thanks for reporting/fixing!

from rss-parser.