{"action":"created","issue":{"url":"https://api.github.com/repos/haugene/docker-transmission-openvpn/issues/292","repository_url":"https://api.github.com/repos/haugene/docker-transmission-openvpn","labels_url":"https://api.github.com/repos/haugene/docker-transmission-openvpn/issues/292/labels{/name}","comments_url":"https://api.github.com/repos/haugene/docker-transmission-openvpn/issues/292/comments","events_url":"https://api.github.com/repos/haugene/docker-transmission-openvpn/issues/292/events","html_url":"https://github.com/haugene/docker-transmission-openvpn/issues/292","id":252506589,"number":292,"title":"Encrypted password in DockerEnv breaks updatePort.sh","user":{"login":"RJSzynal","id":5617631,"avatar_url":"https://avatars0.githubusercontent.com/u/5617631?v=4","gravatar_id":"","url":"https://api.github.com/users/RJSzynal","html_url":"https://github.com/RJSzynal","followers_url":"https://api.github.com/users/RJSzynal/followers","following_url":"https://api.github.com/users/RJSzynal/following{/other_user}","gists_url":"https://api.github.com/users/RJSzynal/gists{/gist_id}","starred_url":"https://api.github.com/users/RJSzynal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RJSzynal/subscriptions","organizations_url":"https://api.github.com/users/RJSzynal/orgs","repos_url":"https://api.github.com/users/RJSzynal/repos","events_url":"https://api.github.com/users/RJSzynal/events{/privacy}","received_events_url":"https://api.github.com/users/RJSzynal/received_events","type":"User","site_admin":false},"labels":[{"id":146134431,"url":"https://api.github.com/repos/haugene/docker-transmission-openvpn/labels/enhancement","name":"enhancement","color":"84b6eb","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-08-24T07:04:31Z","updated_at":"2017-10-15T19:14:53Z","closed_at":null,"author_association":"NONE","body":"When using an encrypted password in the DockerEnv file it is also encrypted in the `/config/transmission-credentials.txt` file which then means the incorrect credentials are used in `transmission/updatePort.sh` on lines 77, 89, and 93. This causes the following output in the container log:\r\n```STARTING TRANSMISSION\r\nCONFIGURING PORT FORWARDING\r\nTransmission startup script complete.\r\nThu Aug 24 07:38:14 2017 Initialization Sequence Completed\r\nGenerating new client id for PIA\r\nGot new port 30166 from PIA\r\ntransmission auth required\r\nUnexpected response: <h1>401: Unauthorized</h1>Unauthorized User1NjIwODBDSm16Vy5WMA==\r\nUser-Agent: transmission-remote/2.92 (14714)\r\nHost: localhost:9091\r\nAccept: */*\r\nAccept-Encoding: deflate, gzip\r\n\r\nUpdate UFW rules before changing port in Transmission\r\ndenying access to 51413\r\nRule updated\r\nallowing 30166 through the firewall\r\nRule added\r\nUnexpected response: <h1>401: Unauthorized</h1>Unauthorized User1NjIwODBDSm16Vy5WMA==\r\nUser-Agent: transmission-remote/2.92 (14714)\r\nHost: localhost:9091\r\nAccept: */*\r\nAccept-Encoding: deflate, gzip\r\n\r\nChecking port...\r\nUnexpected response: <h1>401: Unauthorized</h1>Unauthorized User1NjIwODBDSm16Vy5WMA==\r\nUser-Agent: transmission-remote/2.92 (14714)\r\nHost: localhost:9091\r\nAccept: */*\r\nAccept-Encoding: deflate, gzip\r\n```\r\n\r\nThe current fix is to use a plain text password but this is not great from a security standpoint.\r\nThis caveat should be added to the readme.md as using an encrypted password is supported by transmission in it's settings.json"},"comment":{"url":"https://api.github.com/repos/haugene/docker-transmission-openvpn/issues/comments/336734301","html_url":"https://github.com/haugene/docker-transmission-openvpn/issues/292#issuecomment-336734301","issue_url":"https://api.github.com/repos/haugene/docker-transmission-openvpn/issues/292","id":336734301,"user":{"login":"haugene","id":1038094,"avatar_url":"https://avatars3.githubusercontent.com/u/1038094?v=4","gravatar_id":"","url":"https://api.github.com/users/haugene","html_url":"https://github.com/haugene","followers_url":"https://api.github.com/users/haugene/followers","following_url":"https://api.github.com/users/haugene/following{/other_user}","gists_url":"https://api.github.com/users/haugene/gists{/gist_id}","starred_url":"https://api.github.com/users/haugene/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/haugene/subscriptions","organizations_url":"https://api.github.com/users/haugene/orgs","repos_url":"https://api.github.com/users/haugene/repos","events_url":"https://api.github.com/users/haugene/events{/privacy}","received_events_url":"https://api.github.com/users/haugene/received_events","type":"User","site_admin":false},"created_at":"2017-10-15T19:14:53Z","updated_at":"2017-10-15T19:14:53Z","author_association":"OWNER","body":"You're right, the updatePort.sh does not support encrypted passwords. We're using `transmission-remote` to set the port, and if it supports encrypted passwords the script could be fixed to support it as well. If not, it should be in the README. Any pull requests on this is welcomed :)"}}
{"actor":{"display_login":"haugene"}}