Werkzeug: Das Parsen des Werkzeug-Benutzeragenten ist für das Motorola One Macro-Android-Telefon deaktiviert
Ich habe dies in einer Flask-App bemerkt, aber das verwendet werkzeug, um Useragent-Informationen zu analysieren, und sah das gleiche Verhalten sowohl im Flask-Anforderungsobjekt als auch im eigenständigen werkzeug-Useragent-Parser.
Es sieht so aus, als würde das Vorhandensein des Teilstrings "mac" in "macro" (dem Gerätemodell) ihn abwerfen. Ich bin jedoch der Meinung, dass dies durch das klare "Linux" und / oder "Android 9" außer Kraft gesetzt werden sollte.
>>> from werkzeug.useragents import UserAgent
>>> UserAgent("Mozilla/5.0 (Linux; Android 9; motorola one macro) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/84.0.4147.111 Mobile Safari/537.36").platform
'macos'
Ich würde erwarten, dass platform für diese Useragent-Zeichenfolge "android" ist.
Umgebung: Linux
- Python-Version: 3.7.3
- Werkzeugversion: 1.01
mhelmetag
Alle 8 Kommentare
Das Problem ist, dass UserAgentParser die Plattform findet, indem es naiv nach Schlüsselwörtern sucht.
Hier ist das relevante Code-Snippet:
platforms: Any = (
(" cros ", "chromeos"),
("iphone|ios", "iphone"),
("ipad", "ipad"),
(r"darwin|mac|os\s*x", "macos"),
("win", "windows"),
(r"android", "android"),
...
In Ihrem Fall findet "darwin|mac|os\s*x" eine Übereinstimmung, bevor "android" durchsucht wird. Nach dem, was ich von Benutzeragenten-Headern gesehen habe, können sie stark variieren. Vielleicht kann jemand, der mehr Erfahrung hat, bessere Regex-Muster vorschlagen. Ich mache mir Sorgen, dass unsere einfache Keyword-Suche unzureichend ist und dass es weitere unbemerkte Fehler wie Ihren geben könnte.
jcorvino
am 4. Okt. 2020
Anstatt dies intern zu tun, können Sie möglicherweise eine Abhängigkeit von der ua-Parser- Bibliothek hinzufügen. Ich habe eine Überprüfung des Projekts durchgeführt und es basiert auf einem ziemlich beliebten sprachübergreifenden Kern, der einen sehr großen Testsatz enthält . Die Python-Version unterstützt eine Vielzahl von Python-Versionen und der Code selbst sollte ziemlich stabil sein.
JavaScriptDude
am 25. Dez. 2020
Ich biete eine PR an, es sei denn, Sie möchten eine Pause einlegen. Scheint eine gute Lösung zu sein.
mhelmetag
am 28. Dez. 2020
Ich würde gerne einen Sprung drauf machen. Vielen Dank!
JavaScriptDude
am 28. Dez. 2020
Rad! Vielen Dank!
mhelmetag
am 28. Dez. 2020
Ich bin froh, eine PR in Betracht zu ziehen, die die Übereinstimmungsregeln anpasst, um zu korrigieren, wie dieser Agent übereinstimmt, aber wir werden keine Abhängigkeit dafür hinzufügen.
davidism
am 29. Dez. 2020
Das ua-parser -Paket findet den geeigneten Benutzeragenten mithilfe von Regex-Mustern (ähnlich wie bei unserem Ansatz, aber offensichtlich gründlicher). Ihre Regex-Muster nehmen ~ 5500 Codezeilen ein .
Ich denke, es ist klar, dass wir kein vollständiges UserAgentParser anbieten können, ohne ua-parser als Abhängigkeit hinzuzufügen oder Teile ihres Codes in Werkzeug aufzunehmen.
Alternativ können wir der Readme-Datei und / oder den Dokumenten eine Nachricht hinzufügen, um zu erklären, dass unser UserAgentParser grundlegend ist und in vielen Szenarien nicht funktioniert.
jcorvino
am 29. Dez. 2020
Ich habe # 1989 geöffnet, um diese Plattformzeichenfolge zu adressieren, aber es ist alles andere als eine vollständige Lösung für den Rest des Gesprächs hier.
tonydelanuez
am 2. Jan. 2021
Verwandte Themen
golf-player
·
10Kommentare
alexgurrola
·
5Kommentare
davidism
·
9Kommentare
d42
·
6Kommentare
sorenh
·
4Kommentare
Hilfreichster Kommentar
Anstatt dies intern zu tun, können Sie möglicherweise eine Abhängigkeit von der ua-Parser- Bibliothek hinzufügen. Ich habe eine Überprüfung des Projekts durchgeführt und es basiert auf einem ziemlich beliebten sprachübergreifenden Kern, der einen sehr großen Testsatz enthält . Die Python-Version unterstützt eine Vielzahl von Python-Versionen und der Code selbst sollte ziemlich stabil sein.