GWicke has uploaded a new change for review.
https://gerrit.wikimedia.org/r/94308
Change subject: Only set cookie header on API requests when one was passed in
......................................................................
Only set cookie header on API requests when one was passed in
We don't send Vary: Cookie headers ourselves, but it seems to be well possible
that our Varnish config (by using various MW defaults) varies on cookies by
default. That would explain the cache misses on selser that we are seeing in
production.
Change-Id: Id28070fce871c51b80a9c443cf4bdd790e903449
---
M js/lib/mediawiki.ApiRequest.js
1 file changed, 5 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Parsoid
refs/changes/08/94308/1
diff --git a/js/lib/mediawiki.ApiRequest.js b/js/lib/mediawiki.ApiRequest.js
index 8ac266b..676e0c5 100644
--- a/js/lib/mediawiki.ApiRequest.js
+++ b/js/lib/mediawiki.ApiRequest.js
@@ -560,12 +560,16 @@
timeout: 60 * 1000, // 60 seconds: less than 100s VE timeout so
we still finish
headers: {
'User-Agent': userAgent,
- 'Cookie': env.cookie,
'Connection': 'close',
'x-parsoid-request': 'cache'
}
};
+ if (env.cookie) {
+ // Forward the cookie if set
+ this.requestOptions.headers.Cookie = env.cookie;
+ }
+
if (!options.evenIfNotCached) {
// Request a reply only from cache.
this.requestOptions.headers['Cache-control'] = 'only-if-cached';
--
To view, visit https://gerrit.wikimedia.org/r/94308
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id28070fce871c51b80a9c443cf4bdd790e903449
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Parsoid
Gerrit-Branch: master
Gerrit-Owner: GWicke <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
