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

Remove table for e2e and add an e2eMangledName column #169

Conversation

dantti
Copy link
Contributor

@dantti dantti commented Feb 6, 2018

This tries to fix #156 but it seems a lot more work has to be done

If the code was not complex enough syncing two tables
already started to give UNIQUE constrains errors on
simple sync operations, this also adds initial support
remote delete of an encrypted file
@dantti
Copy link
Contributor Author

dantti commented Feb 7, 2018

@rullzer can you please take a look as well?

@tcanabrava
Copy link
Contributor

@rullzer - please take a look at this as I'm currently unable to review

@rullzer rullzer self-requested a review February 9, 2018 14:28
@rullzer
Copy link
Member

rullzer commented Feb 9, 2018

I have it on my todo

@tcanabrava tcanabrava merged commit 9911707 into nextcloud:clientSideEncryptionV3 Feb 12, 2018
@tcanabrava
Copy link
Contributor

I found the time to review and it seems correct.

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.

3 participants