configparser.InterpolationSyntaxError if there is percent (%) sign in feed url #4

New issue

Closed

opened 2023-11-15 11:54:48 +00:00 by MrClon · 2 comments

MrClon commented

2023-11-15 11:54:48 +00:00

(Migrated from codeberg.org)

ConfigParser raise exception if there is % it feed url (like https://example.org/feed%20name.rss).

configparser.InterpolationSyntaxError: '%' must be followed by '%' or '('

It can be bypassed by this patch to control.py

80c80
<         parser = ConfigParser()
---
>         parser = ConfigParser(interpolation=None)
233c233
<         parser = ConfigParser()
---
>         parser = ConfigParser(interpolation=None)

Or just by mention in docs that user should replace % by %% in config

`ConfigParser` raise exception if there is `%` it feed url (like https://example.org/feed%20name.rss). ``` configparser.InterpolationSyntaxError: '%' must be followed by '%' or '(' ``` It can be bypassed by this patch to `control.py` ``` 80c80 < parser = ConfigParser() --- > parser = ConfigParser(interpolation=None) 233c233 < parser = ConfigParser() --- > parser = ConfigParser(interpolation=None) ``` Or just by mention in docs that user should replace `%` by `%%` in config

marvin8 commented

2023-11-16 20:59:20 +00:00

(Migrated from codeberg.org)

Thanks... I'll add this to the next release... (No date planned as of yet :)

marvin8 commented

2023-11-18 06:10:33 +00:00

(Migrated from codeberg.org)

On the other hand, it would probably faster for you to create a pull request against the wip branch. I am a little ways away from making any changes myself.

On the other hand, it would probably faster for you to create a pull request against the `wip` branch. I am a little ways away from making any changes myself.