Improve HTTP logging

* Log HTTP debug lines to the "http" channel instead of wfDebug()
* Add the ability to do debug logging to MultiHttpClient
* Add a static method Http::createMultiClient() which provides a
  MultiHttpClient configured similarly to the way individual requests
  are configured, respecting the wiki's $wgHTTPTimeout and writing debug
  logs.
* In EtcdConfig, pass the logger instance through to the MultiHttpClient
  backend.

Change-Id: Ic5bdcb0cae95d7b3715ab5261758be082751c3ff

Localisation updates from https://translatewiki.net.

Change-Id: I4b723b8eac7d7df85781495c396db74ea0021dbc

Localisation updates from https://translatewiki.net.

Change-Id: Ia4e0aad6e4f8ee1f1cba85cf72c77d83bffae7f3

Merge "resourceloader: Add filename to validateScriptFile cache key"

Merge "resourceloader: Simplify validateScriptFile() with getWithSetCallback"

resourceloader: Add filename to validateScriptFile cache key

* Add fileName to cache key to fix T52919. The cached parsed error
  message contains the filename, this should be part of the cache
  key as otherwise two identical user scripts may report the same
  error message, including " on line X of page Y" where Y is whichever
  of the two pages first created the cache entry.

* Make the cache key global instead of per-wiki. There is no need
  for this to be per-wiki.

Bug: T52919
Change-Id: I6c2718c53be7f6384a6486a4a8718ae7f423d216

Merge "ApiQueryRevisions: Don't be so tricky handling rvstartid/rvendid"

Localisation updates from https://translatewiki.net.

Change-Id: I411ee26a2f6dd7199c42bc3a8e70fde852255ca7

resourceloader: Simplify validateScriptFile() with getWithSetCallback

* Simplify by using early return and getWithSetCallback.
* Add TTL (previously indefinite, now 1 week).

Bug: T52919
Change-Id: Ic95ba392cdb3bcc8081c77d2c2a3240548bed366

ApiQueryRevisions: Don't be so tricky handling rvstartid/rvendid

If someone strangely supplies 0 as an id, the tricky version confuses
that with null and winds up making the DB layer throw an exception. Just
do it more straightforwardly.

Bug: T165100
Change-Id: Id8376d419e7758c0bf92ad50dd7b7ac283a73101

Refactor filter-specific logic into group models

Make the view model more generic and ready to accept other types
of filtering, like namespaces and tags, by handing off the responsibility
and logic of the parameterizing and filter relationships to the groups.

Change-Id: I9c333719e89088d96d044d72ddb9c39feeeb68ca

RCFilters UI: Add a placeholder to SavedLinksListWidget when it's empty

No longer hide the button when the menu is empty, but instead
show the placeholder item when the *model* is empty.

Bug: T164861
Change-Id: I96e5e375de5f35946663042f6731d7b69e53308b

RCFilters UI: Add tooltip to saved links items

Bug: T164861
Change-Id: Ic7c38c59c53d4d83b959eff36d4b1d198bb654ea

Merge "Pager class for filtering by date range"

Merge "RCFilters: Treat saved query with multiple spaces as empty"

Merge "RCFilters: Actually read/write highlight parameter"

Pager class for filtering by date range

New abstract class RangeChronologicalPager to provide shared date range
filtering capability (with unit tests)

I18n msgs to use as common labels for date range inputs

Expose some ReverseChronologicalParser::getDateCond logic so we can
convert year/month date filters to datestamp date filters

Bug: T120733
Change-Id: I65fdc00368f406f5fa2492600e95e07ce442c165

Merge "Add X-Content-Dimensions support to DjVu"

Merge "Factor out changetag name validation check"

Merge "RCFilters UI: Add down indicator to SavedLinksListWidget button"

Merge "RCFilters UI: Don't remove removed saved links twice"

RCFilters: Treat saved query with multiple spaces as empty

Because it is empty, even if there's multiple spaces. Also, if the
input is invalid and we didn't save the query, don't close the
popup.

Bug: T165113
Change-Id: I30e68b0b90e2a4077fb4068c1462d40b03cbfe8c

RCFilters: Actually read/write highlight parameter

The highlight parameter enables/disables the highlight view. It was
not properly updated to the URL, and a separate typo made it not update
the model correctly.

Bug: T165107
Change-Id: Ie711d4d08fb3eeffca50c2953016e02e2620f898

Merge "Gate option to save RC filters to default false"

Merge "mw.Upload.Dialog: Define .static.name"

Merge "Check for null before calling a textResults method to (fixes T165091)"

Merge "Fix comment typo in MimeMagic.php"

Merge "benchmarks: Convert benchmarkHooks to use Benchmarker"

Gate option to save RC filters to default false

Change-Id: I1c9e382c265a097b5eea07b529ec542a60562307

Fix comment typo in MimeMagic.php

Change-Id: I6343e773c9b9988683b2b28cbbde8fc71359376f

Check for null before calling a textResults method to (fixes T165091)

Bug: T165091
Change-Id: I6fd23eb5fa04049fd7a44faae76f77fae23f5202

Localisation updates from https://translatewiki.net.

Change-Id: I6d7f1e58cbbc0adbdd159ab7d37f2f22a8fd73d6

mw.Upload.Dialog: Define .static.name

Bug: T164999
Change-Id: I0e5d11bdeac1ea60caaa146041057e7a9faf351c

Merge "Handle proxy-based TLS when placed in front of Swift"

Add X-Content-Dimensions support to DjVu

Bug: T150741
Change-Id: I4a3bae9bf056a7ba332f2f0a330697cdf59b4d04

RCFilters UI: Add down indicator to SavedLinksListWidget button

Bug: T164861
Change-Id: I23c602e9bcfbbe8f39f3e3efc4a5482490699214

RCFilters UI: Don't remove removed saved links twice

The second removal also didn't remove the right thing.

Change-Id: Iab32ae2e19e2b19995b88fbfd2e84a36dbda1bac

Merge "Fix usage of $db->nextSequenceValue()"

Factor out changetag name validation check

This way it can be called by AbuseFilter without also getting caught in
the "already exists" checks. And possibly it could also be used in the
future to validate input passed to ChangeTags::addTags()

Bug: T92956
Change-Id: Ic5d754323cbfd2c2b54c4df1245767946ebb1821

Check for index to exist before accessing it.

This fixes "Notice: Undefined index: quality ..."

Bug: T164983
Change-Id: If72c40aeaf0ee5727e21f9fe4dac92c12436ce41

Fix usage of $db->nextSequenceValue()

The return value from the method is only suitable for passing to
$db->insert(). To get the inserted ID, you need to call $db->insertId()
even if $db->nextSequenceValue() returned non-null.

Bug: T164900
Change-Id: I6beb6243ccb9425372623307ef23ae6571ce8c0d

Merge "Avoid postgres CategoryMembershipChangeTest failures"

Localisation updates from https://translatewiki.net.

Change-Id: Iaaf6784416df518129277b41020f308d951c32b9

Avoid postgres CategoryMembershipChangeTest failures

Strip out the redundant /32 CIDR suffix from the IP.

Bug: T75174
Change-Id: If7738ab46c72babb51d3c306ce749a9ccfe34740

Merge "Make LocalPasswordPrimaryAuthenticationProviderTest use TS_MW timestamp conversion"

Merge "Fix some postgres test failures"

Merge "Make loadWatchedItem() coerce the DB timestamp to TS_MW"

Merge "Avoid assuming a user with ID 0 exists in ApiMainTest::testAssert"

Merge "OOUI-ificiation of TablePager nav"

Handle proxy-based TLS when placed in front of Swift

Force the URL for storage operations to use https if the authentication
URL is already forced to use https. This avoids following an http URL
based on the response of a Swift proxy unaware that a terminator placed
in front of it was used for the authentication.

Bug: T160616
Change-Id: Ia6c8c99ebb38d5828773b2f7aa8cf14ced6bdfde

Merge "Add collation for Bashkir (ba)"

Add collation for Bashkir (ba)

This is based on a numeric uppercase collation. Bashkir characters
will be remapped to the private use area for the purpose of sorting.

Bug: T162823
Change-Id: I65f1af0b57ff6ded7d464e39efd401f178a3519e

Make LocalPasswordPrimaryAuthenticationProviderTest use TS_MW timestamp conversion

Bug: T75174
Change-Id: I217a8e1847a21d54434693fafa7f618ec5840876

Avoid assuming a user with ID 0 exists in ApiMainTest::testAssert

If the load() triggered by User method calls fails, then mId becomes 0
which means there is no "user" right set in getAutomaticGroups().

Bug: T75174
Change-Id: I2d719e4b96c0142e9d408aa2d4f7c5e7a767a754

Fix some postgres test failures

Bug: T75174
Change-Id: If9812ae301f0af84fa012e2e980b92036c1b29b2

Make loadWatchedItem() coerce the DB timestamp to TS_MW

Bug: T75174
Change-Id: Ie72bd17b424ba857d0b4a22227c9cffad0853050

Update OOjs UI to v0.21.3

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

Change-Id: I53696e023197ae4dd85605d6615e9faceda05398

Merge "Add missing addQuotes() to ChangesListSpecialPage"

Merge "Make registerTempTableOperation() detect TRUNCATE operations"

Add missing addQuotes() to ChangesListSpecialPage

Bug: T75174
Change-Id: I34a2a3e8661edff68b115e349baddcec0bd14629

Store original media dimensions as additional header

For storage repos that support headers (such as Swift), this will store the original
media dimensions as an extra custom header, X-Content-Dimensions.
The header is formatted to minimize its length when dealing with multipage
documents, by expressing the information as page ranges keyed by dimensions.

Example for a multipage documents with some pages of different sizes:
X-Content-Dimensions: 1903x899:1-9,11/1903x873:10

Example for a single page document:
X-Content-Dimensions: 800x600:1

Bug: T150741
Change-Id: Ic4c6a86557b3705cf75d074753e9ce2ee070a6df

Make registerTempTableOperation() detect TRUNCATE operations

Bug: T145947
Change-Id: I6094624fc594b23b1df6107f7ac1c3d93f609ebc

Localisation updates from https://translatewiki.net.

Change-Id: I13d63f8e59ea5eea6b3a3760075d754064c5fea0

Merge "Allow callback functions for creating jobs"

Allow callback functions for creating jobs

$wgJobClasses can now specify a factory function for creating a job,
instead of a class to be instantiated directly. This makes it possible
to inject services in a job constructor, and register a factory function
that calls the constructor with default services.

This follows Ieb85493a7765 and Ia2107dc5af78, which introduced factory
functions for API modules and special pages.

Change-Id: I0461e59da2a8fa6681e3b1fcdfc38bfed7f3ac32

Merge "Revised styling of sister-search sidebar."

Merge "Change delimiter for multiple namespaces and tags"

Change delimiter for multiple namespaces and tags

It's ; for namespaces (since that is generally what we're using,
e.g. for STRING_OPTIONS), and | for tags.  I would have
preferred them to all be consistent, but there is one revision
that has a ;, and this seems better than modifying the DB.

Comma is already used as a delimiter between separate parameters
(e.g. hideanons and namepsace).

Also, fix multiple namespaces for parseParameters and add test
cases.

Bug: T164132
Bug: T164133
Change-Id: Iad061e0bc17e3522a3f5d330ac2c8bf9cf0e614f

Merge "MessageCache: Avoid 'contributions' replicas"

Merge "Prettify MessageCache::loadFromDB()-small query"

Merge "ApiQueryRevisions: Restore use of rvstartid/rvendid as a tiebreaker"

Follow-up I5cede8: fix JS error when user has no filters saved

Change-Id: Ie7de070925874d2c29e185a90d65a0b0a72f07cd

Followup I5cede8: Make the SavedQueries popup larger

The labels and popup are now slightly wider.

Bug: T151994
Bug: T164128
Change-Id: I64bd496ade241c800fcb0bb12023486e7d601a23

Followup I5cede8: Fixup SavedQueries styling and event

Includes small styling fixes to saved queries and the ability
to update the label of an existing matched query if we rename
it.

Bug: T151994
Bug: T164128
Change-Id: Ic46b082cbfc3462f1d61895bcfc7c545902b2d1a

Merge "RCFilters UI: Add a 'saved queries' quick filters feature"

RCFilters UI: Add a 'saved queries' quick filters feature

Bug: T151994
Bug: T164128
Change-Id: I5cede87633147736d3b4ee5b8ea178ae21bd441f

Revised styling of sister-search sidebar.

Various improvements to the sister-search sidebar:

- using WM project favicons
- scoping CSS specific to sister-search sidebar
- making sister-search items more compact

Bug: T160724, T158938
Change-Id: I2794121ab83cbd4e2b8868150e4d61db376fa63b

Localisation updates from https://translatewiki.net.

Change-Id: I001225f92a48f46a6a9418958bd8603c2d3f403d

ApiQueryRevisions: Restore use of rvstartid/rvendid as a tiebreaker

If rvstartid/rvendid are used with revisions that have the same
timestamp, we should omit any revisions with the same timestamp but
out-of-range IDs.

Bug: T163532
Change-Id: I6ba5e0724945668dad55eec0d09cb7d3fe08e347

MessageCache: Avoid 'contributions' replicas

On WMF wikis, the partitioning of the revision table on the
'contributions' replicas makes the query here perform really poorly.
Specify 'api' as a hack for now to avoid those replicas.

This query happens once per day per wiki, plus when someone edits the
MediaWiki namespace, so it shouldn't be much additional load.

Bug: T164666
Change-Id: I5ae74d1702144f6475e9cfb13effc43389d66233

Prettify MessageCache::loadFromDB()-small query

Adjust the call so it uses JOIN...ON syntax rather than comma joins.

Bug: T164666
Change-Id: I93c8569620d543e9507d0adf1283443915f50d4b

Merge "RC Filters: Single boolean filters are not contradictory"

Enable extensions to allow software-specific tags to be added by users

Bug: T121880
Bug: T121872
Change-Id: I6e054cfc065c8d8ae7d20a518b1963cd0551c5c2

Merge "Fix a couple of issues with cleanupInvalidDbKeys.php maint script"

Merge "RC Filters: Rephrase learner level message"

Merge "Remove "(WMDE)" from my name in .mailmap"

Merge "SpecialRecentchanges: Add class="mw-recentchanges-toplinks" to the "top links""

RC Filters: Rephrase learner level message

Bug: T164611
Change-Id: I5cd7b392fc89402e358084c8153bf8c2b9f1abd2

RC Filters: Single boolean filters are not contradictory

When all boolean filters of a group are selected
(for example: hideanons && hideliu), we consider the selection
unproductive and set them all to false.

Some legacy filters are alone in their group
(for example: ORES hideDamaging). They should not
be considered.

Bug: T164625
Change-Id: Ibb6721ccdfb226b3baac7775c30af230c68309e7

Remove "(WMDE)" from my name in .mailmap

No other WMDE employee does have this. I think this is supposed to be
the "canonical name", which means it should not contain such extra
information, right?

Change-Id: Ic5bbb53d650843dfb627baa4c7508d88eba03729

API: Convert rvstartid/rvendid to timestamps for query

We tried something like this once before, but reverted it because it was
an unintended behavior change (see T98467). This time it's intended, we
need it for query optimization.

The behavior changes here are:
* rvstartid/rvendid is exactly equivalent to specifying rvstart/rvend
  with the corresponding revisions' timestamps.
* If the revision for rvstartid/rvendid is not found in the database, an
  error will be thrown.

This will pull timestamps from deleted revisions, i.e. the `archive`
table. While this is technically an information leak (that some revision
ID exists as a deleted revision and the time the revision was made),
it's minor and in line with the information revealed in Tool Labs thanks
to T51088.

Bug: T163532
Change-Id: Ida64a377c38b3553aa82ac754d80e8f898caf6c5

Merge "Have Title::get(Next|Previous)RevisionID sort by timestamp"

Have Title::get(Next|Previous)RevisionID sort by timestamp

Revision IDs are usually increasing as timestamp increases, but not
always. Callers almost certainly want next/previous timestamp when the
two differ.

This also takes care of a minor bug in the nearby getFirstRevision()
where it'll choose an arbitrary tied revision ID if there were multiple
revisions made in the same second.

Bug: T4930
Bug: T163532
Bug: T159319
Change-Id: Iab2060a0ad5e45edbaa0ff36e863cb014b8e876f

Wrap parser output in <div class="mw-parser-output">

This will allow CSS to target just the parser output, without also
accidentally targeting the edit form, diff tables, and so on.

Bug: T37247
Change-Id: If4eb5bf71f94fa366ec4eddb6964e8f4df6b824a
Depends-On: I330c6aa4aaee045614b1801ed34bc9e03be69650
Depends-On: I52a518fa44e017841fe78474012cd69823e0a41d

Merge "Add DatabaseUpdater::modifyTable"

Merge "CREDITS: remove duplicate entry for Nik Everett"

Localisation updates from https://translatewiki.net.

Change-Id: I2b3b0cb604caa001245786be38730195e242e621

Merge "Add single angle quotation marks and  to special characters"

CREDITS: remove duplicate entry for Nik Everett

Change-Id: I8bc4bcecd12aa4dc41e1826896dbe89104958125

Merge "Update CREDITS"