The proposal to merge lp:~breunigs/duplicity/amazondrive into lp:duplicity has
been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~breunigs/duplicity/amazondrive/+merge/309084
--
Your team duplicity-team is subscribed to branch lp:duplicity.
Ah yes. After a successful oauth Amazon redirects the user to some page, which
contain the needed credentials in the query string. That page needs to be
whitelisted in Amazon's backend, and they only allow to whistelist httpS://bla
URLs. With the sole exception of http://localhost or
Review: Approve
hey Stefan,
out of interest. what is meant by
"
109 +# NOTE: Amazon requires https, which is why I am using my
domain/setup
110 +# instead of Duplicity's. Mail me at stefan-duplic...@breunig.xyz
once it is
111 +# available through https and I will
Review: Needs Information
hey Stefan,
wrt.
1.
can be disregarded if the other backend will be replaced completely
2.
yepp.. overlooked that. don't mind my comment there.
3. and 4.
same as 1.
additionally..
there were concerns when adding the Dropbox backend wrt. the trademark so it
was added
1.
Sure, but the product is called "Amazon Drive", so I oppose "acd" as an
abbreviation. ad, adrive or amazondrive work though. Which one would be most
preferable? I also believe the acdcli backend should be removed, so
"compatibility" to that backend's name is not a good argument for me. See
Stefan Breunig has proposed merging lp:~breunigs/duplicity/amazondrive into
lp:duplicity.
Requested reviews:
duplicity-team (duplicity-team)
For more details, see:
https://code.launchpad.net/~breunigs/duplicity/amazondrive/+merge/309084
As announced on the mailing list, here's the patch for
6 matches
Mail list logo