Merge "Avoid 'message' in log context in AuthManager"

Merge "wgRateLimits: Add configuration option to ignore 'noratelimit' right"

Add missing array_keys() call to scaleLoads()

Bug: T147240
Change-Id: I110f43b5990fe42b2d7f1ec960a55a1ae4aa4253

Improve Database::factory() $driver validation

Change-Id: Ie034e6f2f94f0f1731dbe2468f075297d5a171a2

Document that RequestContext::setTitle's $title can be null

Change-Id: Ibd004fd79c374533128574fb1bf5d3edbd82a17d

Merge "Move most of MWCryptRand into libs"

Move most of MWCryptRand into libs

Dependency-inject the MW-specific parts of MWCryptRand via
MediaWikiServices into a generic CryptRand class that is in libs/.

Note that this removes the wfGetAllCallers() debug logging from
generate().

Change-Id: I9742735c266ee69fb247199d3c553cd2ad2a3987

wgRateLimits: Add configuration option to ignore 'noratelimit' right

It may be reasonable to ignore the 'noratelimit' right granted to a user
when perfuming some rate limit checks. As an example, a rate limit check
on failed authentication attempts should not be bypassed.

Add an optional '&can-bypass' configuration option for each
$wgRateLimits action that can be set to false to disable checking
User::isPingLimitable(). This bypasses both 'noratelimit' and
$wgRateLimitsExcludedIPs exclusions.

Depends-On: Iacdd1719d5f08eca91de0a35c0042ffee2136f34
Change-Id: Ia3add8bbbab0307f036e9b77e752c382da3a0d04

Merge "Move MWCryptHash into libs/"

Merge "Set default load monitor class in LBFactoryMulti"

Set default load monitor class in LBFactoryMulti

Bug: T147240
Change-Id: I6d1c86e714d537f43b01bbf9581032aec2a4e59b

Merge "http: Use Psr\Log instead of wfDebug*"

Merge "Split HttpFunctions.php into separate files"

Merge "Remove pre-5.5 compat from MultiHttpClient"

Merge "Restore Content-Language header change from e53ff931a"

Merge "Revert "Localisation updates from https://translatewiki.net.""

Revert "Localisation updates from https://translatewiki.net."

Broke the tests.

This reverts commit 17ef4586787be33bf00980c42516bed6643656b3.

Bug: T147245
Change-Id: Ic5db3a4243c9543e12852faecf4f2425b16bab39

Avoid 'message' in log context in AuthManager

Logstash merges the log context into the main metadata (where
'message' is the log message) and ends up overwriting the message.

Bug: T145133
Change-Id: I27f221b0f1f7203e93d1b92119dc584ba8526f5b

Implement scaleLoads() in LoadMonitorMySQL

* Recently pooled servers will get more traffic as
  their internal caches warm up by getting queries
* Also let LoadMonitor support options for flexibility
  The "loadMonitor" config field now takes in array in
  LoadBalancer.
* Make LoadMonitor the default instead of LoadMonitorNull.

Change-Id: I53b89b0c25bdcc30deec3f8b502fb14479c53ae8

Localisation updates from https://translatewiki.net.

Change-Id: Ib172a5e6cb38c20cf4149e1735272c27192bc8b6

Update OOjs UI to v0.17.10

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.17.10

Change-Id: I4faf83e301417ef5721a81a4a69890854e6c266b

Merge "action=history should return 404 if there is no such page"

Merge "Fix Database IDEA warnings about callable parameters"

Merge "mw.feedback: Use standard OOjs UI dialog error handling"

Restore Content-Language header change from e53ff931a

This was lost in a rebase at some point.

Change-Id: I46f376a82205a5c99b98c9e971f9e9d7868ce9fb

Fix Database IDEA warnings about callable parameters

Change-Id: Ia9ccc86bb9cbd78b5890ceaedce8cca8d6bb7fe4

Merge "Remove cruft from interwiki.list and interwiki.sql"

Merge "Enable deprecation warnings for WikiPage::getText()"

Merge "Update wikimedia/php-session-serializer to v1.0.4"

Merge "Add primary key to change_tag and tag_summary tables"

Remove cruft from interwiki.list and interwiki.sql

Specifically, remove docbook, gej, jargonfile, lugkr, ppr, sourcewatch,
tejo, uea, and webseitzwiki, all of which are not in the interwiki map,
per https://phabricator.wikimedia.org/T146852#2682115 . Also remove
mediazilla.

Bug: T146852
Change-Id: I04c1e10f989eccc60a0c968617f795fd0e6c6309

Merge "Add "from" to MediaWiki:Search-redirect"

Special:NewPages - add <span> for "originally created as..."

Add span tag around the old title line in the Special New Page.

Bug: T146145
Change-Id: I7578e94284710250eab9f23dbef2777b58031faa

Add "from" to MediaWiki:Search-redirect

Bug: T129941
Change-Id: If8871fe85b1c1c8202895c964d47ef250a1ed110

Update wikimedia/php-session-serializer to v1.0.4

github.com/wikimedia/php-session-serializer/compare/v1.0.3...v1.0.4

Change-Id: Ia0045329f4680a6f7c6acc9d886ae54ccf49c83d
Depends-On: I18ec214b6446f559e018673797efcab70523de39

Merge "Add a class to the <a> element of the interlanguage link"

Move MWCryptHash into libs/

Remove the single wfDebug() call that was making this class MW specific,
someone can log the return value of MWCryptHash::hashAlgo() if they'd
like to know the specific implementation being used.

Change-Id: Ibb7ead7594edab7861631046dd8316daab613401

http: Use Psr\Log instead of wfDebug*

MWHttpRequest::factory() will pass in a logger to move the dependency up
to the factory instead of individual functions.

Change-Id: I4e428f060c90ef49cb3acb3e3dceab64bd952330

Split HttpFunctions.php into separate files

Change-Id: I30d5f77de08e42ec43baf5722f40d9205c4bde67

Remove pre-5.5 compat from MultiHttpClient

Change-Id: I5f9f1a94c0207a14c0d5cc77292aa04af4ac6470

Merge "Detect/use APCu properly"

Merge "Update composer/semver to 1.4.2"

Add primary key to change_tag and tag_summary tables

Based heavily on 43e386ca16411096bbbd7f14f9f4e15c5e268fe7.

Bug: T123225
Change-Id: I33480f4016812259700979f1145099744bb451d4

MWException: restore prior render() logic

Partially revert 00bee02 by restoring the prior MWException::report()
logic. MWExceptionRenderer does not handle all of the MWException
subclasses that somehow modify the default render() behavior. Notable
among these is Flow\Exception\FlowException and its subclasses which
modify the `$wgOut` global to divert HTML output to alternate locations.

Bug: T147122
Change-Id: Ibe3cadca229ce21ed0a3a2482433e3a22b5f5646

Merge "Exception rendering fixes"

Detect/use APCu properly

In PHP 5.5 and above, userland APC caching moved to an extension

Bug: T140587
Change-Id: Ie0871776cd7e67838471a4fe95451cf4164079b7

Localisation updates from https://translatewiki.net.

Change-Id: I0afe820944452c3089c953449ae3415333783f45

Update composer/semver to 1.4.2

https://github.com/composer/semver/compare/1.4.1...1.4.2

Change-Id: I0a83a8568c9a74adbf3a26373ecb90578396c564
Depends-On: I46158f128edb14a5b45f3a8918acb1a7e15935e4

Use wikimedia/wait-condition-loop

Since the WaitConditionLoop class was first introduced in 1.28 (current
master), no back-compat alias is added.

Bug: T146256
Depends-On: Ia84774d83da79fea1e167fe065c69549981f753b
Change-Id: Ibd4f15c87105b8caccbd1f661b74b6efa012b77f

Merge "Fix SectionProfileCallback type error"

Exception rendering fixes

* Actually use MWExceptionRenderer::AS_RAW. Use this after
  an error is thrown while trying to pretty render the original
  error. This is how this case was originally handled before.
* Do not show the google form or file cache in CLI mode.

Change-Id: I130499753efbf8b4d6d254ea36bacb2473952c1b

LegacyLogger: Be consistent about converting log levels to int

Normalize $level arguments to integers early to avoid confusion later
about what type the argument is.

Change-Id: I7305c0514f4383ec552afea2deb0e3fc6ba3fe8a

Fix wfLogDBError channel redirect logic

Change-Id: I1249f9a097625c2fab6601059177594e5f3a21fc

Fix SectionProfileCallback type error

Avoids "Argument 1 passed to Profiler::scopedProfileOut() must
be an instance of ScopedCallback, SectionProfileCallback given"

Change-Id: I92713de71df9722d8a5d7e5cd04460aff71cf096

Merge "database: Rename LBFactoryMW to MWLBFactory"

MWExceptionHandler: Restore delegation to MWException::report

Follow up to 00bee0297. Many MWException subclasses override
MWException::report to do things like special logging and setting the
HTTP response status code. We need to keep calling those methods until
MWExceptionRenderer knows how to handle all of them.

Bug: T147098
Change-Id: I2c90e2d9e9b019357458c7e14a3d602b591c6f5b

database: Rename LBFactoryMW to MWLBFactory

It's no longer an LBFactory subclass (since 5d4b009cf), and the MW-prefix
seems more natural for this class.

The class has only existed since for about 2 weeks (since 0e5cd18b7) and
is not used outside MediaWiki core across Wikimedia Git.

Change-Id: I34be982b5d10ad03e062033da9c40b4a01665289

Replace deprecated `constructive` with `progressive`

Replacing deprecated `constructive` mediawiki.UI CSS class and OOjs UI
flag  with `progressive`.

Bug: T146923
Change-Id: I524b9722ee49692c55bb1f97d34d8a28068716ee

mw.feedback: Use standard OOjs UI dialog error handling

Previously, we'd close the original dialog on failure and display a
new one, which had the unfortunate effect of losing the user's input
(which presumably was not saved due to the failure).

Bug: T147056
Change-Id: I8893658807f6a584d6db0a84bc508d87ef508c7e

Merge "Bidi-isolate revisions count in Special:Import"

Merge "Use MAX_LOCK_TTL in RedisLockManager to avoid premature lock expiration"

Merge "Cleanup MemcLockManager and move it to /libs"

Pass interwiki option to SearchEngine

Currently interwiki result are always computed even if the client does
not request them.
Pass the option via featureData to allow SearchEngine not to run
interwiki searches when not needed.

Bug: T147003
Change-Id: I55067bc7e5d926a3e37f4d24fb8e085bd252c376

Merge "Move ChronologyProtector up a directory instead of being by itself"

Use MAX_LOCK_TTL in RedisLockManager to avoid premature lock expiration

App servers should not lower the redis structure TTL of locks set
from CLI scripts, which is what would typically happen before if
they touched the same keys. Instead, keep a high lock record TTL
and rely on the logical TTLs for expiry semantics.

Change-Id: I2948369845c47d2682a85dd0564673f3b813be83

Cleanup MemcLockManager and move it to /libs

* Remove wf* function and ObjectCache dependencies.
* Use the base class session field.
* Lower physical lock structure TTL and move it to the base
  class as a constant.
* Resolve TODO about acquiring mixed lock types in one pass.
* Only mark servers down for 30 seconds in case of long-running
  scripts.

Change-Id: Icd4be407e599524cc620975d27e85666d2532b95

Bidi-isolate revisions count in Special:Import

Change-Id: I3aacc10dd76aba9f4315db284a4bafe629219bc1

Merge "Remove test for <math> which fails when Extension:Math is installed"

Merge "Don't skip tests which lack non-tidy output"

Merge "JpegMetadataExtractor: Allow empty segments"

Move ChronologyProtector up a directory instead of being by itself

Change-Id: Ie4859a24fb22e5816718a7d1d9444316b1f444c4

MovePage: Check ContentHandler::canBeUsedOn()

If the old content model cannot be used on the new page title, the page
move should be disallowed.

Change-Id: I6078a4cde5a76a050dddedf59634d800765cbabf

JpegMetadataExtractor: Allow empty segments

A segment which consists only of the marker and length value
(equal to 2, the length of the value itself) appears to be
perfectly valid, and is ignored by every image viewer I tested.

Bug: T147015
Change-Id: I3124c587ccb1c457df25fd5bf7a47feab9312a38

Merge "Make multi-write backend "template" config work as expected"

Localisation updates from https://translatewiki.net.

Change-Id: Id78eb74f69b0450cdf09a39a41c057b888c143fa

Make multi-write backend "template" config work as expected

* FileBackendGroup now applies the usual config for these sub-backends,
  the same config get() applies.
* Also make sure FileBackendMultiWrite::concatenate() uses the status wrapper.

Bug: T146904
Change-Id: I1e9b5027dbac11ea9484cd16851e5db998574429

Enable deprecation warnings for WikiPage::getText()

Deprecated since 1.21.

Bug: T145750
Change-Id: I226458e480bc44d4223bf7b271173bc27902f5ba

Merge "Do not run exact db match if offset is > 0"

RELEASE-NOTES for wikimedia/scoped-callback

Change-Id: I197028956f2dc7f0b1edcbcf4ba02687440309a7

Use wikimedia/scoped-callback

The ScopedCallback class was moved into a separate library. This updates
all callers to use the namespaced version, and provides a
backwards-compatibility class wrapper under the old name.

Bug: T146258
Change-Id: I2dd0a66fe2f510f26bdfef6b0a975c1beb3fd93c
Depends-On: Iea0c40bdd7776372ccf72db8f088a2abaa4d3721

Merge "Move DatabaseBase to a class alias for type-hints"

Make WikiPage::getAutosummary emit deprecation warnings.

Bug: T146164
Change-Id: Ibe76af8af3686c028e54923b9ce01b93af3032ab

Revert "Add the main stash, WAN, and server caches to MediaWikiServices"

 $ php maintenance/eval.php
 > $wgMessageCacheType = CACHE_ACCEL;

 > wfGetMessageCacheStorage();
 PHP Warning:  Illegal offset type in isset or empty in includes/objectcache/ObjectCache.php on line 93
 PHP Warning:  Illegal offset type in isset or empty in includes/objectcache/ObjectCache.php on line 125
 PHP Notice:  Array to string conversion in includes/objectcache/ObjectCache.php on line 133
 Caught exception InvalidArgumentException: Invalid object cache type "Array" requested. It is not present in $wgObjectCaches.
 #0 includes/objectcache/ObjectCache.php(94): ObjectCache::newFromId(Array)
 #1 includes/objectcache/ObjectCache.php(287): ObjectCache::getInstance(Array)
 #2 [internal function]: ObjectCache::getLocalServerInstance(Array)
 #3 includes/objectcache/ObjectCache.php(181): call_user_func('ObjectCache::ge...', Array)
 #4 includes/objectcache/ObjectCache.php(137): ObjectCache::newFromParams(Array)
 #5 includes/objectcache/ObjectCache.php(94): ObjectCache::newFromId(3)
 #6 includes/GlobalFunctions.php(3430): ObjectCache::getInstance(3)
 #7 maintenance/eval.php(78) : eval()'d code(1): wfGetMessageCacheStorage()
 #8 maintenance/eval.php(78): eval()
 #9 {main}

This reverts commit 6d99fa58247a1de7eaf9a79a061f0b4207154072.

Change-Id: I4d756eb6f66318a12d8cbbc1ba546397be8a29bf

Merge "Emit deprecation warnings for Article::doEdit and WikiPage::doEdit"

Move DatabaseBase to a class alias for type-hints

Change-Id: Ib6bbbd751358cbc2f2d2767c80721d3d8f142c0d

Merge "registration: Improve license-name validation"

Merge "Replace DatabaseBase => Database in more places"

Merge "Make LBFactory return type docs more specific to avoid IDEA warnings"

Remove test for <math> which fails when Extension:Math is installed

And add a test for <mw:toc>

C. Scott asked for this test to be added in CR on
Ib48b91238c3eddd6a86b62, but it was never actually executed prior to
Ic840678b851d6fc472b834 due to a TestFileReader bug. The nuisance it
causes now becomes apparent.

As was noted at the time, it is unnecessary because Tidy is not a
sanitization pass, <math> is not whitelisted in the Sanitizer. C. Scott
said "I was just concerned that the particular means used to hide the
mathml from tidy might also hide it from the sanitizer." I don't think
that's plausible, but we can test for it anyway by trying to pass
<mw:toc> elements through the Sanitizer.

Change-Id: I5928e6f31d8b80ef384a1c0e6bacfa789cac9770

Don't skip tests which lack non-tidy output

Previously some 60 tests which only had tidy output were silently
removed from the test list and were not executed. Perhaps miraculously,
they all pass (if Math is disabled).

Also, throw an error if a test lacks any kind of result section. Fix one
test which accidentally lacked a result section.

Change-Id: Ic840678b851d6fc472b834cde3d8c83359f1c272

Merge "Add the main stash, WAN, and server caches to MediaWikiServices"

Merge "Disallow css attr() with url type"

Merge "API: Add generator non-continuation data"

Replace DatabaseBase => Database in more places

Change-Id: If37a7909056bf2c31a8228cbc84f0fbbf5f1c517

Make LBFactory return type docs more specific to avoid IDEA warnings

Change-Id: I5fa6ed87e35f3b8ee17e576faf5d00763229dd4b

Merge "resourceloader: Disable ChronologyProtector to speed up wfGetLB()"

Add the main stash, WAN, and server caches to MediaWikiServices

Also removed unused getLocalServerInstance() b/c $fallback logic.

Change-Id: Ifa5f798de10783741a7b079f22d283bb9cb7f4c0

Merge "rdbms: Lazy-init DatabaseDomain::getId()"

rdbms: Lazy-init DatabaseDomain::getId()

Follows-up 847b91b.

Noticed that in Xenon profiles, about 25% of Database::__construct
time is spent in array_map, via DatabaseDomain::convertToString.

* Database::__construct
  * DatabaseDomain::__construct/convertToString/array_map
  * DatabaseDomain::newUnspecified
    * DatabaseDomain::__construct/convertToString/array_map
  * DatabaseMysqlBase::open
    * mysqli::*

Change-Id: Ib5ae7d1d03b27df504a7e5f552e36a0ca6fcf97e

Merge "Parser: Allow `<s>` and `<strike>` in table of contents"