Last active
December 29, 2021 20:42
-
-
Save webdados/a7702e588070f9a1cfa12dff89b3573c to your computer and use it in GitHub Desktop.
Do not show WordPress 5.4 default WP blue favicon
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
<?php | |
//Inspiration: https://gist.github.com/florianbrinkmann/a879099f3d28c5e1d64f2aeea042becf | |
add_action( 'do_faviconico', function() { | |
//Check for icon with no default value | |
if ( $icon = get_site_icon_url( 32 ) ) { | |
//Show the icon | |
wp_redirect( $icon ); | |
} else { | |
//Show nothing | |
header( 'Content-Type: image/vnd.microsoft.icon' ); | |
} | |
exit; | |
} ); |
The changeset https://core.trac.wordpress.org/changeset/13205 shows an additional header line header('Content-Length: 0');
Is this not necessary?
I don't have a clue.
Looks like this is not recommended anymore after rfc2616 is followed by rfc7230
https://datatracker.ietf.org/doc/html/rfc7230#section-3.3.2
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The changeset https://core.trac.wordpress.org/changeset/13205 shows an additional header line
header('Content-Length: 0');
Is this not necessary?