This will allow analysis of the impact of google chrome user-agent deprecation.
Considered headers: Sec=CH-UA*, sent as a key-value string
Description
Description
Details
Details
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| profile::cache::kafka::webrequest: Log Sec-CH-UA* headers | operations/puppet | production | +1 -1 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | • DAbad | T295073 <Org-Wide Impact> Google Chrome User-Agent Deprecation Impact | |||
| Resolved | kostajh | T242825 Deal with Google Chrome User-Agent deprecation | |||
| Resolved | kostajh | T299397 Measure user-agent client hints already sent in browsers requests | |||
| Resolved | • EChetty | T299401 VarnishKafka to propagate user agent client hints headers to webrequest |
Event Timeline
Comment Actions
Change 755435 had a related patch set uploaded (by Phuedx; author: Phuedx):
[operations/puppet@production] profile::manifests::cache::kafka::webrequest: Log Sec-CH-UA* headers
Comment Actions
@JAllemandou: @elukey highlighted that we (Data Engineering and other stakeholders) should agree on the names for these headers as they're added to the JSON blob that varnishkafka produces. For example:
- X-Client-IP is added as ip
- X-Cache-Status as cache_status
- X-Analytics as x_analytics
I can't talk to what's supported by varnishkafka but I'll go ahead and make the new additions consistent with the above while we discuss naming here, i.e.
- Sec-CH-UA will be added as ch_ua
- Sec-CH-UA-Mobile as ch_ua_mobile
- Sec-CH-UA-Platform as ch_ua_platform
Comment Actions
Mentioned in SAL (#wikimedia-analytics) [2022-01-26T10:07:27Z] <btullis> btullis@cumin1001:~$ sudo cumin 'O:cache::upload or O:cache::text' 'disable-puppet btullis-T296064-T299401'
Comment Actions
Change 755435 merged by Btullis:
[operations/puppet@production] profile::cache::kafka::webrequest: Log Sec-CH-UA* headers