Opened 15 months ago

Closed 15 months ago

Last modified 15 months ago

#20137 closed defect (bug) (fixed)

Security Warning when customizing a theme

Reported by: mattrude Owned by: koopersmith
Priority: normal Milestone: 3.4
Component: General Version: 3.4
Severity: normal Keywords: has-patch
Cc: matt@…

Description

When a site using https, when a user chooses 'customize theme' on the themes page (version 3.4-alpha-20032) a security warning is displayed stating that some content will not be display via a secure connection.

Looking deeper, the previewed theme dose not honor "define('FORCE_SSL_ADMIN', true);" and is out put via an insecure connection.

Attachments (1)

20137.patch (619 bytes) - added by ocean90 15 months ago.

Download all attachments as: .zip

Change History (10)

  • Milestone changed from Awaiting Review to 3.4

comment:2 follow-up: ↓ 4   ocean9015 months ago

And the current preview doesn't have a warning?

  • Keywords has-patch added

20137.patch should work for links from get_permalink or wp_enqueue_script but not for hardcoded links.

Just a s/http/https/ could break things.

comment:4 in reply to: ↑ 2   mattrude15 months ago

  • Cc matt@… added

Replying to ocean90:

And the current preview doesn't have a warning?

Correct, in 3.3.1, no error is displayed when previewing a theme under the same setup.

  • Owner set to koopersmith
  • Status changed from new to assigned
  • Resolution set to fixed
  • Status changed from assigned to closed

In [20051]:

Theme Customizer: Load the preview iframe with the same scheme as the admin to prevent security warnings. props ocean90. fixes #20137, see #19910.

comment:7 follow-up: ↓ 9   nacin15 months ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Is there a reason for get_home_url( null, '/', $scheme ) rather than home_url( '/', $scheme )?

  • Resolution set to fixed
  • Status changed from reopened to closed

In [20057]:

Theme Customizer: Use home_url instead of get_home_url. props nacin, fixes #20137, see #19910.

comment:9 in reply to: ↑ 7   koopersmith15 months ago

Replying to nacin:

Is there a reason for get_home_url( null, '/', $scheme ) rather than home_url( '/', $scheme )?

Survey says... “No.” [20057]

Note: See TracTickets for help on using tickets.