Make WordPress Core

Context Navigation

← Previous Ticket
Next Ticket →

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#31735 closed defect (bug) (fixed)

Emoji: Use `set_url_scheme` for CDN's URL

Reported by:	extendwings	Owned by:	pento
Priority:	normal	Milestone:	4.2
Component:	Emoji	Version:	4.2
Severity:	normal	Keywords:	emoji
Cc:		Focuses:	performance

Description

Currently, wp_staticize_emoji() forces all sites to use HTTPS for CDN (even if they are using HTTP!)
I think CDN's URL scheme should be relative.

With this fix, HTTP-user may access the site faster!

Attachments (1)

WordPress_master...shield-9_emoji_cdn_scheme.diff.txt (1.1 KB ) - added by extendwings 11 years ago.

Download all attachments as: .zip

Change History (6)

@extendwings
11 years ago

Attachment WordPress_master...shield-9_emoji_cdn_scheme.diff.txt added

#1 @extendwings
11 years ago

Keywords has-patch added

This ticket was mentioned in Slack in #core by pento. View the logs.

11 years ago

#3 @pento
11 years ago

Owner set to pento
Resolution → fixed
Status new → closed

Emoji: Set the emoji image protocol with set_url_scheme(), instead of defaulting to HTTPS.

Props extendwings.

Fixes #31735.

#4 @pento
11 years ago

Keywords emoji added; has-patch removed
Milestone Awaiting Review → 4.2
Type enhancement → defect (bug)

#5 @pento
11 years ago

Component Formatting → Emoji

Note: See TracTickets for help on using tickets.

Download in other formats: