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
Disable reset password link #4809
Conversation
@nickvergessen, thanks for your PR! By analyzing the history of the files in this pull request, we identified @MorrisJobke, @LukasReschke and @j-ed to be potential reviewers. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that also needs to be added to \OC\Core\Controller\LostController because that is otherwise only cosmetics and not really a good security measure.
Codecov Report
@@ Coverage Diff @@
## master #4809 +/- ##
============================================
- Coverage 54.21% 54.21% -0.01%
- Complexity 22147 22148 +1
============================================
Files 1363 1363
Lines 84789 84791 +2
Branches 1322 1322
============================================
Hits 45971 45971
- Misses 38818 38820 +2
|
@LukasReschke done |
Signed-off-by: Joas Schilling <coding@schilljs.com>
41f9168
to
0828df5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and works 👍
Fix #4008
Downstream 27676