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

Not able to add nzb through API #166

Closed
Undeadhunter opened this issue Nov 14, 2015 · 3 comments
Closed

Not able to add nzb through API #166

Undeadhunter opened this issue Nov 14, 2015 · 3 comments

Comments

@Undeadhunter
Copy link

Issue seems to be the API call sends out a redirect while CP and others expect a NZB file

Trying to fix the issue on https://github.com/Mirabis/usntssearch/tree/couchpotato-fix

However my Python knowledge is pretty much 0, so I've been messing about with it.

Issue seems to be around beam_localwarp in WarperModule.py it sends out a redirect, so the fix would be to get the file from the provider, and then send it back as a correct nzb file.

However not sure how to do it

@Mirabis
Copy link

Mirabis commented Nov 15, 2015

Does it work with warper disabled? Haven't checked yet

@Undeadhunter
Copy link
Author

Nope. Warper seems to ignore options in settings as well, it warps stuff
anyway also in the normal search if you disable it the api still warps it.
Which is an issue all together, check main page template, it checks for the
value of enc url but then ignores it lol...

That isn't the issue however. It's the way cp expects to get the nzb file.
Megasearch sends cp a redirect request to the website of the newznab you
configure, but cp doesn't follow that link. Thus it just downloads the
redirect html file.

Easiest option is to download the file, and send it to cp through http.

But I need to read up on python requests for that, I have general coding
knowledge but nothing specific to python.

On Sun, Nov 15, 2015, 15:03 Moreno Sint Hill notifications@github.com
wrote:

Does it work with warper disabled? Haven't checked yet


Reply to this email directly or view it on GitHub
#166 (comment)
.

@Undeadhunter
Copy link
Author

Issue resolved, made pull request in @Mirabis his fork

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

2 participants