-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Server on http://
returns https://
links
#51
Comments
Same problem for |
Worth noting that So the right filter in that case would probably be |
@swissspidy yep, i find what |
I'm not sure this can be easily solved. You still have all the links that are stored throughout post content fields that point to Maybe a better approach would be to add a flag |
get_site_icon_url
return link with https
http://
return https://
links
http://
return https://
linkshttp://
returns https://
links
@schlessera we rewrite url in production, but for development mode will be great have valid url |
If you have site on
https
and run php server onhttp
the functionget_site_icon_url
return a link withhttps
.I think the problem here https://github.com/WordPress/WordPress/blob/master/wp-includes/general-template.php#L810, because WordPress should use here
get_site_url()
insteadget_option( 'site_icon' )
.Maybe we can add filter on
get_site_icon_url
to fix it problem or we should fix it in WordPress?The text was updated successfully, but these errors were encountered: