mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #8910 from nextcloud/fix-login-error 

Avoid to leak a user ID that is not a string to reach a user backend
This commit is contained in:
Morris Jobke 2018-04-11 01:06:38 +02:00 committed by GitHub
parent 38961a725f a07f6d46e3
commit 52c0e02bdc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
core/Controller/LoginController.php
View File

@ -141,7 +141,8 @@ class LoginController extends Controller {
* *
* @return TemplateResponse|RedirectResponse * @return TemplateResponse|RedirectResponse
*/ */
public function showLoginForm($user, $redirect_url) { public function showLoginForm(string $user = null, string $redirect_url = null): Http\Response {
if ($this->userSession->isLoggedIn()) { if ($this->userSession->isLoggedIn()) {
return new RedirectResponse(OC_Util::getDefaultPageUrl()); return new RedirectResponse(OC_Util::getDefaultPageUrl());
} }