Page MenuHomePhabricator

Define an official thumb API
Open, MediumPublic

Assigned To
None
Authored By
GWicke
Apr 22 2014, 12:24 AM
Referenced Files
None
Tokens
"Like" token, awarded by Veikk0.ma."Baby Tequila" token, awarded by Imarlier."Love" token, awarded by Liuxinyu970226."Love" token, awarded by bearND."Love" token, awarded by MichaelSchoenitzer."Orange Medal" token, awarded by Krinkle."Love" token, awarded by Fjalapeno."Baby Tequila" token, awarded by Mholloway."Barnstar" token, awarded by mmodell.

Description

See also:


MediaWiki never had an official API for the retrieval of thumbnailed images. Existing thumb URLs are considered private, and are not designed for use as a public API. Instead, clients are supposed to request a URL individually from the action API. This complicates API design and API response caching whenever the response references a thumb, and makes it difficult to dynamically select image size or quality in a client.

The move to content-based addressing could provide a good opportunity to address this by defining a simple and stable thumb API.

Use cases / problem statement

  • Dynamic client-side thumb size / quality selection without extra round-trips: Clients would like to adapt image size and quality to actual network and device characteristics. Most current techniques for lazy-loading and dynamic resource selection are using JS. Having the ability to select the thumb size and quality in this code without extra round-trips would enable interesting performance optimizations.
  • Caching of API responses referencing thumbnails: Many cached API responses (such as summary or related pages) contain references to thumbnails. Currently, there is no way for clients to sanely select thumb sizes, which means that we either need to fragment caches on thumb size, or try to include several sizes in the response. Good caching of API responses is becoming more pressing as high-traffic features like hovercards (see T70860) are built as direct API consumers. The need to negotiate thumb sizes supported by API end points introduces delays, and reduces the design flexibility in clients.
  • Image info caching: Parsoid and other clients like VisualEditor currently need to request image information in order to render thumbnails. Those requests are very common, making imageinfo one of the most-used API entry points. Currently, the request necessarily contains the desired thumb size, which renders caching of imageinfo responses ineffective. If the response was dimension-independent instead, most of these responses could be served directly from caches. This would reduce response latency and load on the API cluster and related infrastructure.

API requirements

  • Simple selection of thumb size and -quality without a need for extra API calls.
  • Avoid cache fragmentation with deterministic URLs.
  • Support for encoding complex options in a uniform and extensible manner, without breaking existing use cases or introducing non-determinism.
  • Optional support for content negotiation (ex: client hints) in the future.
  • Support migrating to hash-based image identification in a later stage.

API proposal: Use query strings

  • /v1/someimage.jpg: Original. Returned by API end points referencing thumbnails.
  • /v1/someimage.jpg?w=220: JPG thumb 220px wide.
  • /v1/someimage.jpg?p=22&w=500: 500px thumb of page 22 in a multi-page document.
  • /v1/someimage.jpg?t=2m30s&w=220: Thumb of a video at 2m30s.
  • /v1/someimage.jpg?lang=fr&w=220: Thumb of an SVG, rendered to a PNG using French texts. We aren't explicitly mentioning the file format, so the client does not need to know that the original is an SVG, or that it is rendered to a PNG image.

Server side requirements:

  • Audit and document the existing query string API in thumb.php. T153497
  • Add strict parameter validation. Each thumb should have only a single URL. Don't allow unknown parameters, and (generally) avoid specifying default values explicitly. Exceptions can be made for page & time offset parameters, where little actual fragmentation is expected, and consistency in the use of the parameter is important.
  • Query string order normalization in Varnish (vmod): T138093

Providing original dimensions to clients

In order to accurately calculate and select thumbnail dimensions, clients need to know the original image's dimensions (where applicable). Using this informations, clients can then construct a unique URL for a given thumbnail size, independent of the constraints it applied to select this size. They can also avoid content jumping around by updating image dimensions to the exact thumbnail dimensions, before the thumbnail has loaded.

Currently, MediaWiki already provides original dimensions in data-file-{width,height} attributes for MediaViewer's benefit. Some API responses referencing thumbnails include the equivalent information in JSON:

image: {
  src: "/2fd4e1c67a2d28fced849ee1bb76e7391b93eb12",
  width: 640,
  height: 480
}

We can either stick to these different formats, or consider unifying this information in the URL, either in query parameters (/someimage.jpg?oh=768&ow=1024&w=100), or separately in a fragment (/someimage.jpg?w=100#oh=768&ow=1024). The latter avoids sending back purely informational parameters to the server.

Pros

  • Familiar query string syntax with wide parsing support.
  • Does not distinguish between advanced & frequently changed properties.

Cons

  • Requires custom ordered query string serialization code for both simple (size, quality) & more complex use cases.
    • Subtlety of ordering requirement (still works) means that users will often ignore it, causing client side cache fragmentation.
  • Need for general query string normalization in Varnish. Weak con, as this would be generally useful.

Options for content negotiation and -selection

We would like to use modern thumbnail formats where this has a benefit for users, but need to make sure that we don't break older clients with insufficient support in the process.

The two main approaches for this content negotiation process are:

a) Server-side HTTP content negotiation, using client supplied headers like accept, and
b) client-side JS explicitly requesting specific formats.

There are pros & cons to either method, and they aren't mutually exclusive. HTTP content negotiation can work without any client-side effort for bitmap formats (ex: Chrome advertises WebM support). Client side JS can add explicit parameters for more fine-grained control, but also has only limited information to base such decisions on.

Either way, the status quo and starting point is to serve widely supported formats (JPG and PNG) by default. We don't need to solve this question right away, and the proposed API is leaving all options for content negotiation open.

Deployment strategy

A change like this isn't complete without a strategy that allows us to roll out new-style thumbs gradually. To avoid performance impacts, all (simple) requests for a thumb of a given size should map to the same cache entries using either URL scheme. To this end, we can roll things out in a way that lets us *rewrite* one URL scheme to the other.

  • New to old style:
    • Simple thumbs (only width parameter specified):
      1. Prefix the image name with the width parameter value followed by 'px-'.
      2. Calculate the MD5 of the image name, and prefix first & first two chars of hex encoding to path (ex: /7/72/).
    • Complex thumbs: Let PHP code handle the request & cache the response separately.
  • Old to new style:
    • Simple thumbs (need to check if we can determine this with a regex):
      1. Extract & strip width parameter
      2. Send a request with a query string to backend
    • Complex thumbs: Let PHP code handle the request & cache the response separately.

Users / apps making assumptions about the current thumb URL format

See T153498.

Migration strategies in Varnish

  • Feasibility of rewriting majority of "simple" thumbnails (no key-value parameters) in Varnish.
  • Feasibility of avoiding redirect latency penalty by resolving redirect responses from thumb service in Varnish.

Small MediaWiki installs

Currently, MediaWiki defaults to serving thumbnails directly from an upload directory. This means that there is no PHP code involved in serving thumbnails. This is good for performance (especially without caching), but also means that on-demand generation & a parameter-based API cannot be supported out of the box. There are several options we can pursue:

  1. Start to serve all thumbs through thumb.php (or an API module). The migration to this would generally be easy (especially with the API), but we would add the overhead of serving thumbs through PHP. Authentication would be supported out of the box. Caching could eliminate the performance issue for higher volume installs, or in an appliance container install that includes Varnish.
  2. Create a new way of supporting direct file serving with 404 handler and storage based on encoded query strings. This requires an advanced web server configuration, and might not be possible with less common web servers.

At this point, the default option would be 1), but we can always optimize the setup with 2). We can provide 2) by default in a container-based distribution solution.

Details

Reference
bz64214

Related Objects

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes
kchapman subscribed.

TechCom is declining because the use case is not current. This needs a new owner and use case.

There is actually one current use case still , which is the idea of having the thumb urls be versioned instead of “current” (ref. Performance-Team), but we intend to address that at a later point with a different proposal.

The current thumbnail URL scheme could easily start including a revision number or sha1 of the original without changing the format.

TechCom is declining because the use case is not current.

Everything listed in the task's description under "Use cases / problem statement" seems current to me. See also T66214#1842437.

Thanks @Anomie my information might be old. Moving to TechCom-RFC Inbox for discussion.

TechCom discussed this at our last meeting. The problem statement is still valid, but that doesn't mean it needs to be kept open as there is currently no resourcing for this. If there is still interest in this issue it could be used as material for a new RFC but the new RFC should contain one problem statement. Note: T149847: RFC: Use content hash based image / thumb URLs has already been broken out into a single issue.

We are moving this to last call to be declined closing on 2018-03-29 at 1 pm PST(21:00 UTC, 22:00 CET

@kchapman we are interested in picking this up in Reading Infrastructure, but haven't been able to get to it. We would still like to do this if we can find some time… I should know more in Q4 about feasibility/timing.

For context: We have lots of client code with work arounds for getting the right sizes of images. So much duplication and bugs. We want to get rid of this code in the client and instead use this much more flexible proposed API.

The problem here is that TechCom is using Phabricator in a different way from the rest of the movement.

The normal way is that you create one task for one concept / task, and multiple groups share that task and do their workflow management in such a way that it does not conflict with that of other groups. That means using projects or workboards (since there can be any number of those but there is only one task status). Declining a task means that it was decided that it should not be done (ie. people should be actively prevented from doing it) because it is a bad idea. Most tasks are not resourced but kept open (or stalled) nevertheless.

So if TechCom insists on their current workflow, it should make it clear that it prefers a different workflow, and for every idea that goes through the TechCom process there should be a separate idea task and a separate RfC task so that the TechCom can decline the RfC task when there is no resourcing, while the idea task can be kept open as something that's potentially still valid; and the existing RfC tasks should be split in two. Or TechCom should change their workflow to match that of everyone else on Phabricator, and use a workboard column or the removal of the TechCom tag or something similar for tracking "rejected without prejudice".

In either case please keep this task open, whether it gets resourced in the near future or not. The use cases described here are still valid, the solution proposed here still captures our best understanding of the solution space, declining it would be confusing and hamper the use of Phabricator as a technical knowledge management system.

@Tgr perhaps I was not as clear as I could have been. The other issue we see is there should be multiple RFCs broken out for that. Perhaps that means this is not an RFC, but an overall task that has RFCs linked to it.

I will bring up the process in the next TechCom meeting.

The other issue we see is there should be multiple RFCs broken out for that. Perhaps that means this is not an RFC, but an overall task that has RFCs linked to it.

I would disagree, I don't think this RfC can be meaningfully broken up. There are a few ideas it mentions but does not actually propose to do (hash-based identification, content negotiation) and those could be removed for clarity, but that's all I can see.

But regardless, even if the task is not a real RfC, that's not a good reason to decline it as a Phabricator task. Probably the RfC project tag should just be removed in that case.

@Tgr we are just putting it in the Declined TechCom-RFC workboard, not in Phabricator as a whole. For reference, this is how we approach declining RFCs now: https://phabricator.wikimedia.org/T184653

TechCom is declining at this time but will be more than happy to discuss further RFCs on this topic in the future (noting that @Fjalapeno mentioned interest in picking this up).

Ugh, I am really sorry, I don't know how I could misread that so badly :( I guess my eye is trained to react to the word "Declined" in Phabricator emails, without reading them properly.

@Jdrewniak shared a pointer to the https://cloudinary.com API which I thought could be a source of inspiration for this task. Be sure to check out their 57 second demo.

This task has been assigned to the same task owner for more than two years. Resetting task assignee due to inactivity, to decrease task cookie-licking and to get a slightly more realistic overview of plans. Please feel free to assign this task to yourself again if you still realistically work or plan to work on this task - it would be welcome!

For tips how to manage individual work in Phabricator (noisy notifications, lists of task, etc.), see https://phabricator.wikimedia.org/T228575#6237124 for available options.
(For the records, two emails were sent to assignee addresses before resetting assignees. See T228575 for more info and for potential feedback. Thanks!)

BBlack subscribed.

The swap of Traffic for Traffic-Icebox in this ticket's set of tags was based on a bulk action for all such tickets that haven't been updated in 6 months or more. This does not imply any human judgement about the validity or importance of the task, and is simply the first step in a larger task cleanup effort. Further manual triage and/or requests for updates will happen this month for all such tickets. For more detail, have a look at the extended explanation on the main page of Traffic-Icebox . Thank you!