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

Jottacloud upload implementation changed to use new api #4715

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

albertony
Copy link
Contributor

@albertony albertony commented Apr 24, 2022

Note: Builds on top of #4699, which adds oauth authentication. Will rebase if/when that gets merged.

This PR changes upload implementation from an old API into the same that current official client (and rclone) uses. The new api has built-in support for deduplication and resuming.

Includes an additional commit that avoids changing the type when using an existing custom device in Jottacloud.

@albertony albertony force-pushed the jottacloud_oauth_allocate branch 3 times, most recently from 1e95b3e to 2118dcd Compare April 25, 2022 06:47
@kenkendk
Copy link
Member

@albertony PR #4699 is now merged.

@albertony
Copy link
Contributor Author

Great news, thank you!
Rebased this PR, but I may need to work on it a bit more, at least do some manual testing, i.e. still work in progress.

@duplicatibot
Copy link

This pull request has been mentioned on Duplicati. There might be relevant details there:

https://forum.duplicati.com/t/jottacloud-error-401-unauthorized/3929/233

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

Successfully merging this pull request may close these issues.

None yet

3 participants