Skip to content

[stable9] Fix decrypt message stable9 #25188

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.

Sign up for GitHub

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

Jump to bottom
Merged
DeepDiver1975 merged 5 commits into from
Jul 1, 2016
Merged

[stable9] Fix decrypt message stable9 #25188

DeepDiver1975 merged 5 commits into from
Jul 1, 2016

Conversation

@DeepDiver1975
Copy link
Member

@DeepDiver1975 DeepDiver1975 commented Jun 20, 2016 •
edited
Loading

backport of #24992

@DeepDiver1975 DeepDiver1975 added this to the 9.0.3-current-maintenance milestone Jun 20, 2016
@mention-bot
Copy link

mention-bot commented Jun 20, 2016

By analyzing the blame information on this pull request, we identified @schiessle, @nickvergessen and @MorrisJobke to be potential reviewers

@PVince81
Copy link
Contributor

PVince81 commented Jun 20, 2016

Not critical => 9.0.4

@PVince81 PVince81 modified the milestones: 9.0.4-next-maintenance, 9.0.3-current-maintenance Jun 20, 2016
@PVince81
Copy link
Contributor

PVince81 commented Jun 20, 2016

👍 still

@davitol
Copy link
Contributor

davitol commented Jun 21, 2016

👎

  1. run command sudo -u www-data ./occ encryption:decrypt later (later is the user's name of an encrypted account)

Expected output: You are about to start to decrypt all files stored in later's account...
Current output: You are about to start to decrypt all files stored in your ownCloud...

@DeepDiver1975
Copy link
Member Author

DeepDiver1975 commented Jun 21, 2016

the cherry-pick is totally fucked up -> 💀

@DeepDiver1975
Originally:
6d8d1b5 
commit 2304e4b
Author: Joas Schilling <[email protected]>
Date:   Tue Jun 7 09:13:11 2016 +0200

    Allow to decrypt user '0' files only
@DeepDiver1975 DeepDiver1975 force-pushed the fix_decrypt_message-stable9 branch from 2304e4b to 6d8d1b5 Compare June 21, 2016 08:44
@DeepDiver1975
Copy link
Member Author

DeepDiver1975 commented Jun 21, 2016

@davitol thx for testing - mind rerunning the tests? THX

@davitol
Copy link
Contributor

davitol commented Jun 21, 2016

👍 WFM now 💃

@PVince81 PVince81 mentioned this pull request Jun 24, 2016
Merged
@guruz
Copy link
Contributor

guruz commented Jun 27, 2016

code looks ok 👍

@DeepDiver1975 DeepDiver1975 merged commit 4ac256e into stable9 Jul 1, 2016
@DeepDiver1975 DeepDiver1975 deleted the fix_decrypt_message-stable9 branch July 1, 2016 09:30
@lock
Copy link

lock bot commented Aug 5, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

4 - To release

Projects

None yet

Milestone

9.0.4

Development

Successfully merging this pull request may close these issues.

7 participants

@DeepDiver1975 @mention-bot @PVince81 @davitol @guruz @SergioBertolinSG