Do not show WordPress 5.4 default WP blue favicon

remove_wp_54_favicon.php

<?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;
} );

Hi! Thanks for making this, it fixed the favicon. However the script itself is showing at the top of the page (in inserted it in the head tags), do you know how to fix that?

This is not intended to be inserted on the header tags. This is a PHP code snippet to be inserted on your child theme functions.php file or in a PHP code snippets plugin.

Thank you!

…

On Wed, May 13, 2020 at 12:10 AM Marco Almeida ***@***.***> wrote: ***@***.**** commented on this gist. ------------------------------ This is not intended to be inserted on the header tags. This is a PHP code snippet to be inserted on your child theme functions.php file or in a PHP code snippets plugin. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <https://gist.github.com/a7702e588070f9a1cfa12dff89b3573c#gistcomment-3302109>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AKIDWEXLHEWGGGTEUX74SNTRRIMUTANCNFSM4M7LSYWA> .

The changeset https://core.trac.wordpress.org/changeset/13205 shows an additional header line header('Content-Length: 0');

Is this not necessary?

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