Just fixed a bug with the "add/byfeedurl" endpoint that would miss on permutation checking when making sure a feed doesn't exist already.

Before adding a url to the `feeds_added` queue table it's checked with various permutations like http/https/endingslash/noendingslash/etc. If the url contained "http" somewhere else in the url body, it would get morphed and not match. Dumb bug.

Thanks to @alberto for reporting it.

cc: @Ben

This fixes it:

· · Web · 3 · 1 · 2

@alberto @Ben I feel a need to be code transparent when I have silly bugs like this. It's therapy. lol.

@dave @alberto @Ben Sounds like to one I reported, that has "http" embedded in the URL .. always 500 response .. got tired of it and set feed as already in PI

@cisene @alberto @Ben Sorry for dropping the ball Chris. I'm sure I got distracted by shiny things. 😢

@dave @alberto @Ben no worries, it got solved

Wonky swedish proverb, "better late than maybe"

Sign in to participate in the conversation
PodcastIndex Social

Intended for all stake holders of podcasting who are interested in improving the eco system