Is there a bug in the RSS trigger causing issues with angle brackets in post titles?

Bugs
21

so in the trigger, its def wrong, ie its reporting the title w/ the code stripped out. it looks like the logic is:

import rss from “…/…/app/rss.app.mjs”;

How do I edit or see that though?

22

Yes the editing trigger code directly in the dashboard is a bit limited, you can’t edit imported files from that view currently.

But you can clone the Pipedream repository which includes this RSS trigger:

23

Specifically here:

Then you can edit this code, and use pd dev new-item-in-feed.mjs to deploy a private version of this source to your account

24

hmm, ok, let me try. i assume if i find the bug, i can report it, maybe issue a pr?

25

Exactly :slightly_smiling_face:

26

The repository is open source, so you can make a PR with your fix and become an official PD contributor

27

i think i have in the past. i believe merge-rss-feeds was one of mine

28

heh yeah and is ee a typo in my comments ;l)

29

Nice. Yup you can make any changes you’d like, change the xml parser package version, add additional props to modify parsing, etc.

30

right now im just testing in RunJS, and the package used, feedparser, is definitely messing up. going to see if its a known issue. i also double checked with a feed validator, and my escaping is kosher

31

boom, open bug

32

and it was intentional - with a workaround, testing that

33

The package.json in the source should list the pinned version of the feedparser package. I wonder if we could just downgrade it for now

34

nah the fix looks safe/easy

35

(err, when i say nah, i mean imo :wink:

36

if i fork and submit a mod w/o testing, would yall hate me? :slightly_smiling_face:

37

I think the component devs will actually test it before merging, that’s how the current flow works

38

ok, i’ll be honest when i submit the pr

39

its a -very- minimal change