Crash bug when quotes are in episode name: about greg

[manolomartinez]

This is now fixed, but I need to think about the best way to sanitize strings in placeholders. Anyway, please confirm that it's now working.

Thanks,
Manolo

[mfeif]

That's not a placeholder, that's the title of a feed entry. Or maybe I don't know what you mean? I presume if it's getting through feedparser, it's valid? I don't think we have to worry about SQL injection or such ;-)

Seems like the 'eval' is the problem? I haven't dug into greg's call tree enough to see why eval is needed.

If I do greg check -f tnyrh and then greg download 17 it works just fine for me:

Downloading Episode 12: Sarah Koenig on "Serial," and a Resilient Poet -- tnyradiohour010816_cms564184_pod.mp3

Does that part work for you? Or is this error only occurring when greg is run via cron?

[mfeif]

It does work with newest git-tip.

[manolomartinez]

That's not a placeholder, that's the title of a feed entry. Or maybe I don't know what you mean?

I meant you get to the offending string to be evaled, i.e.,

"Episode" in "Episode 12: Sarah Koenig on "Serial," and a Resilient Poet"

by taking the line you have in greg.conf (presumably something like filter = "Episode" in "{title}") and substituting the title of the feed entry for "{title}".

The resulting string is then evaluated to get a condition that greg uses to filter by. In the eval step the extra pair of quotes are making greg trip up. That's what I meant by sanitizing.

[manolomartinez]

I think this is solved, right? I will close it now, but do let me know if there's anything that needs to be ironed out.