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

Sending to Transmission is broken #1998

Closed
Wiltons opened this issue Nov 11, 2014 · 9 comments
Closed

Sending to Transmission is broken #1998

Wiltons opened this issue Nov 11, 2014 · 9 comments
Labels

Comments

@Wiltons
Copy link
Contributor

Wiltons commented Nov 11, 2014

Hi all,

I updated my headphones yesterday (from being way out of date) and the sending to Transmission is now broken. I've enabled debug and found the issue:

DEBUG :: Requesting URL via GET method: http://localhost:9091/rpc/transmission/rpc

Tracked it back to this recent change:
6952505#diff-4dde0daad672dbb58ffc1fe433d2bc3c

The host is getting /rpc/ added before checking if there is a port, leading to the above "/rpc/transmission/rpc" instead of the correct "/transmission/rpc"

I'm guessing the changes on line 146 and 154 need to be reverted

@Gandrushka
Copy link

Hi. It seems i've got same problem. I'm trying to send download to transmission and all i get is this

Error getting Session ID from Transmission
Error sending torrent to Transmission. Are you sure it's running?

@Wiltons
Copy link
Contributor Author

Wiltons commented Nov 12, 2014

I have submitted a pull request with a fix

@Jumbosausage
Copy link

I'm getting the same thing

Error getting Session ID from Transmission
Error sending torrent to Transmission. Are you sure it's running?

Other programs such as couchpotato are sending to transmission fine using the same details.

@basilfx
Copy link
Contributor

basilfx commented Nov 12, 2014

#1999 will fix this, when its merged.

As a workaround: change the url to: http://ip:port/transmission/rpc

@basilfx basilfx added the Bug label Nov 12, 2014
@Jumbosausage
Copy link

Hi Bas

Many thanks that's working for me now. Cheers.

On Wed, Nov 12, 2014 at 2:19 PM, Bas Stottelaar [email protected]
wrote:

#1999 #1999 will fix this,
when its merged.

As a workaround: change the url to: http://ip:port/transmission/rpc


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

basilfx added a commit that referenced this issue Nov 12, 2014
@basilfx
Copy link
Contributor

basilfx commented Nov 12, 2014

A fix is available in the develop branch: 548e0a9.

basilfx added a commit that referenced this issue Nov 13, 2014
* develop:
  Release v0.5.1
  Prepare for next release
  Fixed HTML div closing tag. Updated titles
  Fixed typo
  Allow one to disable all tasks
  Allow one to adjust interval settings without restarting.
  Prevent infinite recursion while scanning library.
  Fix for new paths
  Added the real pkg_resources from setuptools 7.0. Should fix #2001
  Fixed contrib scripts. Added *.pyc cleaning scripts
  Check if pyOpenSSL is installed before enabling HTTPS.
  Ignore CSR files
  Fixed SSL too long errors in CherryPy.
  Spaces removal
  Add downgrade functionality (contributed script)
  Use urlparse to parse transmission URL
  Fix for #1998
@basilfx
Copy link
Contributor

basilfx commented Nov 13, 2014

It's in master now.

@basilfx basilfx closed this as completed Nov 13, 2014
@Jumbosausage
Copy link

Thanks Bas, do I need to change it back or can I leave it as is?

On Thu, Nov 13, 2014 at 7:03 PM, Bas Stottelaar [email protected]
wrote:

It's in master now.


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

@basilfx
Copy link
Contributor

basilfx commented Nov 13, 2014

You can leave it as is, both ways will work now :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants