Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: sitescripts/stats/bin/logprocessor.py

Issue 29340823: Issue 3737 - Parse user agent strings from MS Edge (Closed)
Patch Set: Created April 25, 2016, 3:19 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | sitescripts/stats/test/logprocessor.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/stats/bin/logprocessor.py
===================================================================
--- a/sitescripts/stats/bin/logprocessor.py
+++ b/sitescripts/stats/bin/logprocessor.py
@@ -185,16 +185,20 @@
return "Gecko", "%s.%s.%s" % (match.group(1), match.group(2), match.group(3))
else:
return "Gecko", "%s.%s" % (match.group(1), match.group(2))
match = re.search(r"\bCoolNovo/(\d+\.\d+\.\d+)", ua)
if match:
return "CoolNovo", match.group(1)
+ match = re.search(r"\bEdge/(\d+)\.\d+", ua)
+ if match:
+ return "Edge", match.group(1)
+
match = re.search(r"\bChrome/(\d+\.\d+)", ua)
if match:
return "Chrome", match.group(1)
match = re.search(r"\bVersion/(\d+\.\d+)", ua)
if match and re.search(r"\bMobile Safari/", ua):
return "Mobile Safari", match.group(1)
if match and re.search(r"\bSafari/", ua):
« no previous file with comments | « no previous file | sitescripts/stats/test/logprocessor.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld