Open
Bug 1154835
Opened 9 years ago
Updated 3 years ago
whitespace should be stripped out of search engine names
Categories
(Firefox :: Search, defect, P3)
Firefox
Search
Tracking
()
NEW
People
(Reporter: cww, Unassigned)
Details
(Whiteboard: [hijacking][fxsearch])
UITelemetry tracks how often you use various one-off search buttons. This is really useful but it trips me up when the search engine reported has newlines in it (and honestly, whitespace, too). It would be nice not to have to deal with it (and also generally trim whitespace from search engine names as a rule) for example: other- Gbase.com Guitar Search .oneoff
Comment 1•9 years ago
|
||
This is not a problem only for telemetry. The gbase.com engine also causes the tooltip on its one-off button to be larger than other one-off tooltips, and the 'Search <engine>' text from bug 1110767 (the gray bar in the middle of the panel, when hovering a one-off button) also shows excessive whitespace. Fixing this could also help a tiny bit to reduce hijacking, as I think we have observed some hijackers working around our rule of "no engine with the same name as a default engine" by appending some whitespace to the engine name.
Priority: -- → P3
Summary: Strip newlines from Telemetry for one-off search, search engine → whitespace should be stripped out of search engine names
Whiteboard: [hijacking][fxsearch]
Updated•9 years ago
|
Rank: 35
Comment 2•9 years ago
|
||
(In reply to Florian Quèze [:florian] [:flo] from comment #1) > Fixing this could also help a tiny bit to reduce hijacking, as I think we > have observed some hijackers working around our rule of "no engine with the > same name as a default engine" by appending some whitespace to the engine > name. Some also add random punctuation signs, eg '!' or '®'. We should also strip these before duplication checks.
Updated•9 years ago
|
OS: Mac OS X → All
Priority: P3 → P2
Hardware: x86 → All
Version: 35 Branch → Trunk
Comment 3•8 years ago
|
||
(In reply to Florian Quèze [:florian] [:flo] from comment #1) > I think we have observed some hijackers working around our rule of > "no engine with the same name as a default engine" by appending some whitespace > to the engine name. As I understand you have the data to back the "hijackers" claim?
Comment 4•8 years ago
|
||
(In reply to Stefan Plewako [:stef] from comment #3) > (In reply to Florian Quèze [:florian] [:flo] from comment #1) > > I think we have observed some hijackers working around our rule of > > "no engine with the same name as a default engine" by appending some whitespace > > to the engine name. > > As I understand you have the data to back the "hijackers" claim? I had such data at the time I wrote comment 1, yes.
Comment 5•8 years ago
|
||
OK, how did you recognize that it was different from legitimately working around that rule and other bugs or anything like details?
Comment 6•8 years ago
|
||
It's not possible to have thousand of users hand-editing a file and doing the exact same change to the name and submission URL, especially to add an affiliation code in the submission URL.
Comment 7•8 years ago
|
||
(In reply to Florian Quèze [:florian] [:flo] from comment #6) > It's not possible to have thousand of users hand-editing a file and doing > the exact same change to the name and submission URL, especially to add an > affiliation code in the submission URL. Oh, it is possible and only extremely unlikely and where this is coming from? I do not get how 'I think we have observed some hijackers working around our rule of "no engine with the same name as a default engine" by appending some whitespace to the engine name.' becomes editing and becomes hijacking in all cases and not working around 10 year old Mozilla bugs.
Updated•8 years ago
|
Assignee: nobody → standard8
Updated•7 years ago
|
Assignee: standard8 → nobody
Comment 8•6 years ago
|
||
Moving to p3 because no activity for at least 1 year(s). See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
Priority: P2 → P3
Updated•3 years ago
|
Severity: normal → S4
Rank: 35
You need to log in
before you can comment on or make changes to this bug.
Description
•