Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Add .ico to the list of secure file extensions #28

Merged
merged 1 commit into from

2 participants

@derf

At https://chaosdorf.de/, we have an apple-touch-icon and "shortcut icon" and noticed that only the touchicon (.png) is changed to HTTPS, not the shortcut icon which ends in .ico, causing an HTTPS mixed content warning.

This patch should fix that.

@mvied mvied merged commit 0aa3610 into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 17, 2014
  1. @derf
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 2 deletions.
  1. +3 −2 lib/WordPressHTTPS.php
View
5 lib/WordPressHTTPS.php
@@ -72,7 +72,8 @@ class WordPressHTTPS extends Mvied_Plugin_Modular {
'jpg',
'jpeg',
'png',
- 'gif'
+ 'gif',
+ 'ico'
),
'style' => array(
'css'
@@ -613,4 +614,4 @@ public function getDirectories() {
return $scannedDirectories;
}
-}
+}
Something went wrong with that request. Please try again.