The method finds a value for a key in a map-formatted string of format k1=v1;k2=v2... but has a bug when a key is a substring of another one, for instance looking for key preview in the string ...;chrome_preview=...;
Description
Description
Details
Details
Customize query in gerrit
| Title | Reference | Author | Source Branch | Dest Branch | |
|---|---|---|---|---|---|
| Fix Refine webrequest to include bug fix in new jar version | repos/data-engineering/airflow-dags!592 | aqu | T355391_remove_utilities_get_values_for_key_in_refinery_source | main | |
| Fix Refine webrequest to include bugfix in new jar version | repos/data-engineering/airflow-dags!591 | aqu | T355391_remove_utilities_get_values_for_key_in_refinery_source | main |
Related Objects
Related Objects
Event Timeline
Comment Actions
Change 991563 had a related patch set uploaded (by Btullis; author: Btullis):
[operations/puppet@production] Revert "varnish: enrich X-Analytics for browser prefetch / prerender / preview"
Comment Actions
This issue was discovered shortly after this patch was deployed: https://gerrit.wikimedia.org/r/c/operations/puppet/+/981352
It relates to this task: T346463: Identify and label prefetch proxy data in our traffic
We have decided to revert the change to varnish whilst we work on fixing the refinery code.
Comment Actions
Change 991563 merged by Btullis:
[operations/puppet@production] Revert "varnish: enrich X-Analytics for browser prefetch / prerender / preview"
Comment Actions
The change that uncovered this behaviour in refinery has been temporarily reverted, so we should not receive any more of these for the time being.
Comment Actions
Change 992475 had a related patch set uploaded (by Aqu; author: Aqu):
[analytics/refinery/source@master] [WIP] Use str_to_map built-in fct to parse x-analytics-header
Comment Actions
Change 992477 had a related patch set uploaded (by Aqu; author: Aqu):
[analytics/refinery@master] [WIP] Use str_to_map built-in fct to parse x-analytics-header
Comment Actions
Change 992475 merged by jenkins-bot:
[analytics/refinery/source@master] Adopt a more resilient approach to use webrequest x-analytics
Comment Actions
Change 992477 merged by Aqu:
[analytics/refinery@master] Adopt a more resilient approach to use webrequest x-analytics
Comment Actions
aqu opened https://gitlab.wikimedia.org/repos/data-engineering/airflow-dags/-/merge_requests/591
Fix Refine webrequest to include bugfix in new jar version
Comment Actions
aqu merged https://gitlab.wikimedia.org/repos/data-engineering/airflow-dags/-/merge_requests/591
Fix Refine webrequest to include bugfix in new jar version
Comment Actions
Change 992752 had a related patch set uploaded (by Aqu; author: Aqu):
[analytics/refinery/source@master] Fix null pointer error in isPageviewUdf
Comment Actions
aqu opened https://gitlab.wikimedia.org/repos/data-engineering/airflow-dags/-/merge_requests/592
Fix Refine webrequest to include bug fix in new jar version
Comment Actions
Change 992752 merged by Aqu:
[analytics/refinery/source@master] Fix null pointer error in isPageviewUdf
Comment Actions
aqu merged https://gitlab.wikimedia.org/repos/data-engineering/airflow-dags/-/merge_requests/592
Fix Refine webrequest to include bug fix in new jar version