Merge lp:~breunigs/duplicity/amazondrive3 into lp:~duplicity-team/duplicity/0.8-series

Proposed by Stefan Breunig
Status: Merged
Approved by: Kenneth Loafman
Approved revision: 1155
Merge reported by: Kenneth Loafman
Merged at revision: not available
Proposed branch: lp:~breunigs/duplicity/amazondrive3
Merge into: lp:~duplicity-team/duplicity/0.8-series
Diff against target: 25 lines (+2/-2)
2 files modified
duplicity/backends/adbackend.py (+1/-1)
duplicity/backends/onedrivebackend.py (+1/-1)
To merge this branch: bzr merge lp:~breunigs/duplicity/amazondrive3
Reviewer Review Type Date Requested Status
duplicity-team Pending
Review via email: mp+313872@code.launchpad.net

Description of the change

As reported on the mailinglist, if a space is entered while duplicity asks for the URL, it fails. Since all important spaces are URL encoded anyway, this should be fine even if there are spaces in the URL at all. I also patched it in the onedrive backend, because it must have similar issues.

Cheers
Stefan

To post a comment you must log in.
Revision history for this message
Kenneth Loafman (kenneth-loafman) wrote :

Another one LP missed. Already merged.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'duplicity/backends/adbackend.py'
--- duplicity/backends/adbackend.py 2016-11-06 14:02:54 +0000
+++ duplicity/backends/adbackend.py 2016-12-26 08:07:40 +0000
@@ -147,7 +147,7 @@
147 print ''147 print ''
148148
149 redirected_to = (raw_input('URL of the resulting page: ')149 redirected_to = (raw_input('URL of the resulting page: ')
150 .replace('http://', 'https://', 1))150 .replace('http://', 'https://', 1)).strip()
151151
152 token = self.http_client.fetch_token(152 token = self.http_client.fetch_token(
153 self.OAUTH_TOKEN_URL,153 self.OAUTH_TOKEN_URL,
154154
=== modified file 'duplicity/backends/onedrivebackend.py'
--- duplicity/backends/onedrivebackend.py 2016-06-08 19:06:18 +0000
+++ duplicity/backends/onedrivebackend.py 2016-12-26 08:07:40 +0000
@@ -149,7 +149,7 @@
149 'page the dialog leads to.' % authorization_url)149 'page the dialog leads to.' % authorization_url)
150 print()150 print()
151151
152 redirected_to = raw_input('URL of the blank page: ')152 redirected_to = raw_input('URL of the blank page: ').strip()
153153
154 token = self.http_client.fetch_token(154 token = self.http_client.fetch_token(
155 self.OAUTH_TOKEN_URI,155 self.OAUTH_TOKEN_URI,

Subscribers

People subscribed via source and target branches