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.

Sign up for GitHub

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

Jump to bottom

[BUG]: #1133

Closed
Alistair1231 opened this issue Oct 14, 2023 · 4 comments
Closed

[BUG]: #1133

Alistair1231 opened this issue Oct 14, 2023 · 4 comments
Assignees
@martinrotter
Labels
Component-Plugins-Standard Status-Fixed Ticket is resolved. Type-Defect This is BUG!!!
Milestone
4.5.2

Comments

@Alistair1231
Copy link

Alistair1231 commented Oct 14, 2023 •
edited
Loading

Brief description of the issue

I created a python script to modify feeds and then output the result to stdout. It works great and wihtout issues. This is how I manually add the feed into RSS Guard.
image

When I export an OPML however, I cannot import it afterwards, since it seems to treat my script as an URL and then of course fails. I think that, since I get the same error "protocol error", when I add a feed and forget to select Script as source before pressing "Fetch it now"

How to reproduce the bug?

  1. Add feed that uses a script
  2. Export feed
  3. Try to import the feed

What was the expected result?

That I can import my feeds and use the OPML as a backup

What actually happened?

It failed to apply the source as script and failed to import.

Debug log

time=" 2094.943" type="debug" -> network: Destroying SilentNetworkAccessManager instance.
time=" 2094.944" type="critical" -> core: Cannot fetch medatada for feed: '"C:\ProgramData\miniconda3\python.exe" "C:\RSS Guard\scripts\script.py" "http://[url]" "another argument"' with error: 'protocol error'.

Operating system and version

  • OS: Win 11
  • RSS Guard version: 4.5.1

Example OPML that was exported

<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<opml version="2.0" xmlns:rssguard="https://github.com/martinrotter/rssguard">
  <head>
    <title>RSS Guard</title>
    <dateCreated>Sat, 14 Oct 2023 18:08:37 GMT</dateCreated>
  </head>
  <body>
    <outline description="" text="Ebooks">
      <outline description="[description]" encoding="UTF-8" rssguard:postProcess="" rssguard:xmlUrlType="1" text="[title text]" title="[title text]" type="rss" version="RSS" xmlUrl="[command]"/>
    </outline>
  </body>
</opml>
@Alistair1231 Alistair1231 added the Type-Defect This is BUG!!! label Oct 14, 2023
@martinrotter
Copy link
Owner

Yes, I see the error on source code. Will fix on monday, maybe tuesday.

@martinrotter
Copy link
Owner

Please post a screenshot of how your "import opml" dialog looks like when you import your opml, thanks.

@Alistair1231

martinrotter added a commit that referenced this issue Oct 16, 2023
@martinrotter
couple of fixes for #1133 and working on other parsing things
fe72cd1
@martinrotter martinrotter added this to the 4.5.2 milestone Oct 16, 2023
@martinrotter
Copy link
Owner

likely fixed, wait for 4.5.2 release

@Alistair1231
Copy link
Author

Please post a screenshot of how your "import opml" dialog looks like when you import your opml, thanks.

@Alistair1231

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@martinrotter martinrotter

Labels
Component-Plugins-Standard Status-Fixed Ticket is resolved. Type-Defect This is BUG!!!
Projects
None yet
Milestone
4.5.2
Development

No branches or pull requests
2 participants
@martinrotter @Alistair1231