search

futtta / ao_critcss_aas

Autoptimize power-up to integrate with criticalcss.com
9 stars 1 forks source link

extra checks on URL's sent to criticalcss.com #107

Closed futtta closed 5 years ago

futtta commented 5 years ago

I have however now verified that these requests are coming in this format from the plugin, so can you please take a look as well to see if that's fixable from your end? These are my relevant logs for the last requests for these domains

2019-05-18T12:38:02 critical css request for url http://xyz.abc133.130.126.119:43?ao_noptimize=1 2019-05-22T04:08:02 critical css request for url http://xyz.abcwww.baidu.com:443?ao_noptimize=1 2019-05-24T19:28:17 critical css request for url http://xyz.abcwww.voanews.com:443?ao_noptimize=1

futtta commented 5 years ago

@pocketjoso is this still happening? because I don't think these requests were issued by the plugin, as the plugin uses a native WordPress function to get the site's own domain. so not sure if there anything I can do here really ..

pocketjoso commented 5 years ago

Did you make a change in this regard in another commit Frank?

futtta commented 5 years ago

no, no change, no idea what to change to be honest.

On Tue, Jul 16, 2019 at 9:26 PM Jonas Ohlsson Aden notifications@github.com wrote:

Did you make a change in this regard in another commit Frank?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/futtta/ao_critcss_aas/issues/107?email_source=notifications&email_token=AABMIMMSTY5OCEDSAIV4DGDP7YOHPA5CNFSM4HR5OLS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2B4ODY#issuecomment-511952655, or mute the thread https://github.com/notifications/unsubscribe-auth/AABMIMILWMFCF3JUKP7CZNDP7YOHPANCNFSM4HR5OLSQ .