Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feed URLs are escaped #115

Open
reconman opened this issue Jan 8, 2021 · 1 comment
Open

Feed URLs are escaped #115

reconman opened this issue Jan 8, 2021 · 1 comment

Comments

@reconman
Copy link

reconman commented Jan 8, 2021

Problem

I just started experimenting with this solution so I don't know much. I read through the readme and some past issues and didn't find anything that could help me.

I configured an RSS feed and now nothing happens. Nothing is logged except for the lines below.

And btw, the log file specified below is never used. Everything is only logged to the docker stdout.

Command:

docker run -t \
  --name=transmission-rss \
  --network=mediaNet \
  --env=TZ=Europe/Vienna \
  --restart=always \
  -v /volume1/docker/transmission-rss/transmission-rss.conf:/etc/transmission-rss.conf \
  -d \
  nning2/transmission-rss:latest

Config:

feeds:
  - url: https://nyaa.si/?page=rss&u=DKB0512&q=Yuru%20Camp

server:
  host: transmission
  port: 9091
  tls: false
  rpc_path: /transmission/rpc

log:
  target: ~/transmission-rss/log/transmissiond-rss.log
  level: debug

Log

2021-01-07 23:52:54 (debug) aggregator start
2021-01-07 23:52:54 (debug) aggregate https://nyaa.si/?page=rss&u=DKB0512&q=Yuru%2520Camp
@reconman
Copy link
Author

reconman commented Jan 8, 2021

Actually, do you URL escape the URLs? Is %20 turned into %2520 when searching?

Edit: That was it. Putting a + instead of a %20 into the URL fixed it.

But do you really expect users to modify the URLs to get it working? For example, when you search for special characters and then copy paste the feed URL.

@reconman reconman changed the title Aggregator does nothing? Feed URLs are escaped Jan 8, 2021
totorowechat added a commit to totorowechat/transmission-rss that referenced this issue Jun 4, 2022
nning pushed a commit that referenced this issue Jun 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant