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

Tranmission 2.93 compatability #1099

Closed
Tangbravo221 opened this issue Mar 20, 2018 · 38 comments
Closed

Tranmission 2.93 compatability #1099

Tangbravo221 opened this issue Mar 20, 2018 · 38 comments

Comments

@Tangbravo221
Copy link

No description provided.

@Tangbravo221
Copy link
Author

HI Guys,

This is my first issue, so please forgive any breach of protocol.
Transmission 2.93 was released today (3/19/2018) and doesn't require a username. The current version (5.14) won't authenticate unless there IS a password.

Thanks,

TB

@antekgla
Copy link
Contributor

I don´t understand...
In 2.93 it is possible authenticate with username only (without password)
or without username/password at all?

@AffekeNommu
Copy link

AffekeNommu commented Mar 20, 2018

I am also having the same issue. I can get to my copy of Transmission 2.93 via the web interface with password and no username just fine but I cannot configure Transmission Remote GUI to get to it as it does not save the login credentials if the username is blank.
Yes Transmission 2.93 allows you to login using just the password and leaving username blank.

@Tangbravo221
Copy link
Author

Tangbravo221 commented Mar 20, 2018 via email

@teufelhunden0311
Copy link

teufelhunden0311 commented Mar 20, 2018

It seems the remote gui is busted until an update allows authentication without a username. It's sucks too because my seedbox updated our transmission clients yesterday.

@Tangbravo221
Copy link
Author

Tangbravo221 commented Mar 20, 2018 via email

@antekgla
Copy link
Contributor

antekgla commented Mar 20, 2018

I try to look at it...
...but I dont understand: what is the advantage of not have a Username and only password?
What security problem resolve that?
There any scenario where that is desirable?

edit: for a temporary fix can't you edit your settings.json in transmission daemon and add a username?

@Tangbravo221
Copy link
Author

Tangbravo221 commented Mar 20, 2018 via email

@leonsoft-kras
Copy link
Contributor

@antekgla - there is version 2.93 for windows. We need to install and configure the input of only the password. Next, look under the debugger, what happens in the TRGUI.
Perhaps the problem isn't only in the TRGUI. It may be necessary to modify the RPC protocol in the program.

@Tangbravo221
Copy link
Author

Tangbravo221 commented Mar 20, 2018 via email

@antekgla
Copy link
Contributor

@Tangbravo221 you need to do what your provider said:

  • Log in your seedbox (using putty or another terminal program)
  • Stop the transmission daemon (ask your provider how do that)
  • Edit ~/.config/transmission-daemon/settings.json adding in the rpc-username the username you want (between quotes)
  • Save ~/.config/transmission-daemon/settings.json
  • Start the transmission daemon again.
  • Log in with TransGUI with the new username.

@Tangbravo221
Copy link
Author

Tangbravo221 commented Mar 20, 2018 via email

@PeterDaveHello
Copy link
Member

@Tangbravo221 no offense, but I think maybe less emotions here can help solving the problems, we're all trying to imrpove this project by our own spare time.

@AffekeNommu
Copy link

I have been having a poke around the code and can see that blank username is being used to set the authentication flag in LoadConnSettings in connoptions.pas.
I have added a username in the rpc-username settings in the json and login is working happily for Transmission Remote GUI now.
Really do not know why Transmission chose to default to password and blank username...

@antekgla
Copy link
Contributor

I have been having a poke around the code and can see that blank username is being used to set the authentication flag in LoadConnSettings in connoptions.pas.

Yes I see it but that is not the problem. That line in LoadConnSettings in connoptions.pas only prevent of read the password from ini file if the username is blank and fill the password edit.

I have being debugging and seems a deeper problem. I guess what is not transGUI fault besides a problem in synapse library.., but I am only guessing and I need to investigate a lot more.

@Tangbravo221
Copy link
Author

Tangbravo221 commented Mar 20, 2018 via email

antekgla added a commit that referenced this issue Mar 21, 2018


Solution to an error in Synapse library what exclude all authorization info when user is blank. Now that is done when user AND pass are blank.
@antekgla
Copy link
Contributor

Fixed in fafa368

Was an error in Synapse library what exclude all authentication info when the user is empy.
Now checks user AND pass, if any is not empty the auth info is included.
Also adapted the LoadConnSettings procedure in connoptions.pas to load the password also when the user is empty.

@Tangbravo221 sadly you have to wait to a new release to enjoy this fixes 😢 unless what @PeterDaveHello make a 5.15.1 release 😉

@PeterDaveHello
Copy link
Member

@antekgla not really, we have temporarily builds in every commit :)

@PeterDaveHello
Copy link
Member

@Tangbravo221 what's your operating system? Except Windows, we provide temporarily builds for other platforms that we support.

@Tangbravo221
Copy link
Author

Tangbravo221 commented Mar 21, 2018 via email

@PeterDaveHello
Copy link
Member

@Tangbravo221 hmmm, I'll release v5.15.1 soon for that, as we don't have automatic build for Windows, it takes more time and manual works to release for it.

@Tangbravo221
Copy link
Author

Tangbravo221 commented Mar 21, 2018 via email

@PeterDaveHello
Copy link
Member

Thanks. Hopefully, v5.15.1 will be released within 12 hours.

@Tangbravo221
Copy link
Author

Tangbravo221 commented Mar 21, 2018 via email

@PeterDaveHello
Copy link
Member

@Tangbravo221 due to some reasons, I have to push the release a little bit later, about 6 hours later. Just let you know.

@Tangbravo221
Copy link
Author

Tangbravo221 commented Mar 22, 2018 via email

@PeterDaveHello
Copy link
Member

@Tangbravo221
Copy link
Author

Tangbravo221 commented Mar 22, 2018 via email

@PeterDaveHello
Copy link
Member

@Tangbravo221 you're welcome

@cfpp2p
Copy link

cfpp2p commented Mar 23, 2018

@antekgla @PeterDaveHello
daemon also accepts empty for both in settings.json and authenticates with web client.
I can't authenticate like that with transgui right now.
I don't know what's a use case or security reason but regardless, whether we like this daemon behavior or not.

    "rpc-authentication-required": true, 
    "rpc-password": "", 
    "rpc-username": "", 

fafa368
looks like we can do entirely without cbAuth.Checked and if (User <> '') or (Pass <> '') then

All other usename password configurations function with v5.15.2

thanks for v5.15.2 :)

@PeterDaveHello
Copy link
Member

@cfpp2p I saw that you have a forked version of transgui, maybe you'd like to contribution some codes? :)

@cfpp2p
Copy link

cfpp2p commented Mar 30, 2018

@PeterDaveHello thanks for opportunity, we'll see, time is crazy...

@PeterDaveHello
Copy link
Member

@cfpp2p 😄

@barbequesauce
Copy link

@cfpp2p described my use case nicely. No username/password, instead access is controlled by the rpc whitelist. Hoping this can be addressed...

@antekgla
Copy link
Contributor

@cfpp2p @barbequesauce
What is the point of have Authentication = true and no username and password. Makes no sense...
Make Authentication= false and TransGUI would be connect leaving Authentication checkmark uncheck.

@barbequesauce
Copy link

My understanding was that rpc-authentication-required must be true to enable the whitelists...

@antekgla
Copy link
Contributor

My understanding was that rpc-authentication-required must be true to enable the whitelists...

It not true... You can make Authentication=false and have Whitelist active.
Whitelist on/off is controlled by rpc-whitelist-enabled = true

@barbequesauce
Copy link

Then I must have some other complication... I'll open a separate issue for that.

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

No branches or pull requests

8 participants