Cicalese has submitted this change and it was merged.

Change subject: PluggableAuth::login should call $user->setCookies()
......................................................................


PluggableAuth::login should call $user->setCookies()

Set cookies so that API calls (e.g. those made by VisualEditor) can be properly 
authenticated.

Bug: T140114
Change-Id: Ibfc5a31c01f859103605a3c4878e36ff2e8dedb9
---
M PluggableAuth.class.php
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Cicalese: Verified; Looks good to me, approved
  jenkins-bot: Verified



diff --git a/PluggableAuth.class.php b/PluggableAuth.class.php
index efbbdc5..b3ec2d6 100644
--- a/PluggableAuth.class.php
+++ b/PluggableAuth.class.php
@@ -219,6 +219,7 @@
                                        $new_user = false;
                                        wfDebug( "Authenticated existing user: 
" . $user->mName . PHP_EOL );
                                }
+                               $user->setCookies();
                        } else {
                                wfDebug( "Authentication failure." . PHP_EOL );
                                return false;

-- 
To view, visit https://gerrit.wikimedia.org/r/298489
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibfc5a31c01f859103605a3c4878e36ff2e8dedb9
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/PluggableAuth
Gerrit-Branch: master
Gerrit-Owner: MarkAHershberger <[email protected]>
Gerrit-Reviewer: Cicalese <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to