catchpoint / WebPageTest.agent

Cross-platform WebPageTest agent
Other
212 stars 138 forks source link

WordPress CDN -> Automattic CDN #311

Open igrigorik opened 4 years ago

igrigorik commented 4 years ago

https://github.com/WPO-Foundation/wptagent/blob/638ae7af9ba14dad25bf3cb05011fa102ad6d59d/internal/optimization_checks.py#L210

A8C powers WordPress.com, but they're one of many WP hosting platforms. I'd like to propose that we relabel this to signal "Automattic" instead of "WordPress".

Additionally, we can add the header signal to the list: https://github.com/rviscomi/ismyhostfastyet/blob/master/ttfb.sql#L5

/cc @rviscomi

pmeenan commented 4 years ago

I can do the rename but I'm less comfortable with the header. Where/when does that value show up in the response headers?

I just loaded wordpress.com and looked at a few of the s*.wp.com responses and don't see anything that looks like that. There are "x-ac" and "x-nc" response headers that look like they may be unique and have cache edge node and hit status but I'm not 100% sure they are unique to Automattic.

rviscomi commented 4 years ago

It's in the X-Hacker header value, eg:

x-hacker = If you're reading this, you should visit automattic.com/jobs and apply to join the fun, mention this header.

I also noticed ~1k sites with wpvip.com/careers in the header value. https://techcrunch.com/ is one example.