HomePhabricator

Update patch set 1
4d4d2e63a7bdUnpublished

Unpublished Commit · Learn More

Not On Permanent Ref: This commit is not an ancestor of any permanent ref.

Description

Update patch set 1

Patch Set 1:

Short summary: Overall I'm unsure if this is worth it.

Long:

If we want to do this, we would need to move the cdnjs copy from tools to production.

This is on another domain name, so that means another DNS request and with HTTP 2 another connection. That means on browser cache miss on modern browsers this will result in worse performance. If it would be a cache hit because of sharing it with another domain, then that would improve performance. The WMF production mediawiki installation does not use a shared domain for cachable resources anymore. So we won't have any gain from sharing with those. Overall I'm unsure if this is worth it.

https://www.w3.org/TR/SRI/ dropped the caching option to not trade off privacy, so that won't help.

Though my guess is on unlikely, maybe service workers can be used cross domain in some way to help here?

Patch-set: 1
Reviewer: JanZerebecki <640@e9e9afe9-4712-486d-8885-f54b72dd1951>
Label: Verified=0

Details

Provenance
JanZerebeckiAuthored on Feb 21 2016, 7:47 PM
Gerrit Code Review <gerrit@wikimedia.org>Committed on Feb 21 2016, 7:47 PM
Parents
rWDQG59a354d0f188: Update patch set 1
Branches
Unknown
Tags
Unknown
ChangeId
None

Event Timeline

Gerrit Code Review <gerrit@wikimedia.org> committed rWDQG4d4d2e63a7bd: Update patch set 1 (authored by JanZerebecki).Feb 21 2016, 7:47 PM

Empty Commit

This commit is empty and does not affect any paths.