Skip to content
This repository has been archived by the owner on Dec 3, 2019. It is now read-only.

Fix the token exception classes to avoid infinite loop with getPrevious #71

Merged
merged 1 commit into from
Sep 13, 2018

Conversation

sharidas
Copy link
Contributor

Fix the users token exception class to avoid infinite
loop when getPrevious method is called.

Signed-off-by: Sujith H [email protected]

@sharidas sharidas self-assigned this Sep 13, 2018
@sharidas
Copy link
Contributor Author

Fixes owncloud/core#32684

@sharidas sharidas added this to the development milestone Sep 13, 2018
@sharidas
Copy link
Contributor Author

Backport PR to stable10 : owncloud/core#32690

Fix the users token exception class to avoid infinite
loop when getPrevious method is called.

Signed-off-by: Sujith H <[email protected]>
@sharidas sharidas force-pushed the fix-previousexception-withsetpassword branch from 7ef2301 to bd4206e Compare September 13, 2018 09:56
@sharidas
Copy link
Contributor Author

https://drone.owncloud.com/owncloud/user_management/249/47 failure has nothing to do with the change made in this PR. I will try to restart the build.

@sharidas
Copy link
Contributor Author

Restarting is not helping its failing again at https://drone.owncloud.com/owncloud/user_management/250/47

@PVince81
Copy link
Contributor

hmm, the random "invalid quota value"

Copy link
Contributor

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@phil-davis
Copy link
Contributor

Same quota acceptance tests fail again.
I triggered CI without this change here #72 - let's see if that passes...

@phil-davis
Copy link
Contributor

#72 got the same quota fails, so it is not the fault of this PR.
Issue #73 raised to look into it.

@PVince81 PVince81 merged commit c7e1dec into master Sep 13, 2018
@PVince81 PVince81 deleted the fix-previousexception-withsetpassword branch September 13, 2018 12:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants