sabconnectplusplus icon indicating copy to clipboard operation
sabconnectplusplus copied to clipboard

NZBIndex "Send to SABNZBD" needs to use filename not URL

Open ChrisWeiss opened this issue 11 years ago • 5 comments

I believe this was fixed for some other providers - When using the direct "Send to SABNZBD" link on NZBIndex, it's sending the URL name to SABNZBD, which often contains illegal characters - breaking local linux file operations. It should send the download name instead.

ChrisWeiss avatar Nov 17 '12 23:11 ChrisWeiss

Don't use the context menu option if one click is supported for a site. The context menu option is a last resort, and can't be customized to behave differently on different sites.

Sent from my Galaxy Note II On Nov 17, 2012 5:41 PM, "Chris Weiss" [email protected] wrote:

I believe this was fixed for some other providers - When using the direct "Send to SABNZBD" link on NZBIndex, it's sending the URL name to SABNZBD, which often contains illegal characters - breaking local linux file operations. It should send the download name instead.

— Reply to this email directly or view it on GitHubhttps://github.com/gboudreau/sabconnectplusplus/issues/28.

rcdailey avatar Nov 17 '12 23:11 rcdailey

This is not the context menu - it's the one-click links that sabconnect++ adds to the page. Sorry for not being clear on that. Example - Any of the one-click links on this page fail for me: https://www.nzbindex.com/search/?q=ssl-news&age=&max=25&minage=30&sort=agedesc&minsize=&maxsize=&dq=&poster=&nfo=&hidespam=0&hidespam=1&more=1

ChrisWeiss avatar Nov 17 '12 23:11 ChrisWeiss

I read back over what you said and I apologize, I misunderstood. You were perfectly clear about what you meant to say. Sorry about that!


Robert Dailey

On Sat, Nov 17, 2012 at 5:47 PM, Chris Weiss [email protected]:

This is not the context menu - it's the one-click links that sabconnect++ adds to the page. Sorry for not being clear on that.

— Reply to this email directly or view it on GitHubhttps://github.com/gboudreau/sabconnectplusplus/issues/28#issuecomment-10480546.

rcdailey avatar Nov 18 '12 03:11 rcdailey

FWIW - I found a patch written to fix this issue for one of the other providers back when sabconnect++ was still in google code: http://code.google.com/p/sabconnectplusplus/issues/detail?id=30

ChrisWeiss avatar Nov 18 '12 03:11 ChrisWeiss

Did you try the "nzbindex.com: Use display name instead of NZB filename." option in Sabconnect++ ? Enable or disable it.

gboudreau avatar Nov 18 '12 13:11 gboudreau