Closed
Bug 1818198
Opened 1 year ago
Closed 1 year ago
Do not record credit card form detected event telemetry when the form is not considered as a valid cc form
Categories
(Toolkit :: Form Autofill, defect, P1)
Toolkit
Form Autofill
Tracking
()
RESOLVED
FIXED
112 Branch
Tracking | Status | |
---|---|---|
firefox-esr102 | --- | unaffected |
firefox110 | --- | wontfix |
firefox111 | --- | fixed |
firefox112 | --- | fixed |
People
(Reporter: dlee, Assigned: dlee)
References
(Regression)
Details
(Keywords: regression, Whiteboard: [fxcm-addr-telemetry])
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
dmeehan
:
approval-mozilla-beta+
|
Details | Review |
We record credit card detect event telemetry even when a credit card section is invalid.
Updated•1 year ago
|
Keywords: regression
Comment 1•1 year ago
|
||
Set release status flags based on info from the regressing bug 1801039
status-firefox110:
--- → affected
status-firefox111:
--- → affected
status-firefox112:
--- → affected
status-firefox-esr102:
--- → unaffected
Assignee | ||
Comment 2•1 year ago
|
||
Pushed by dlee@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/363429f0a2eb Record form detect event telemetry only after we're sure this is a valid form r=credential-management-reviewers,sgalich
Comment 4•1 year ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
Comment 5•1 year ago
|
||
The patch landed in nightly and beta is affected.
:dimi, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- If no, please set
status-firefox111
towontfix
.
For more information, please visit auto_nag documentation.
Flags: needinfo?(dlee)
Updated•1 year ago
|
Updated•1 year ago
|
Assignee | ||
Comment 6•1 year ago
|
||
Comment on attachment 9319326 [details]
Bug 1818198 - Record form detect event telemetry only after we're sure this is a valid form r=#credential-management-reviewers
Beta/Release Uplift Approval Request
- User impact if declined: No. This patch only affects how we collect telemetry. Uplift this patch ensure telemetry collected in Fx111 is correct
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Low, this patch only changes when we collect telemetry
- String changes made/needed:
- Is Android affected?: No
Flags: needinfo?(dlee)
Attachment #9319326 -
Flags: approval-mozilla-beta?
Comment 7•1 year ago
|
||
Comment on attachment 9319326 [details]
Bug 1818198 - Record form detect event telemetry only after we're sure this is a valid form r=#credential-management-reviewers
Approved for 111.0b7
Attachment #9319326 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 8•1 year ago
|
||
bugherder uplift |
Assignee | ||
Updated•1 year ago
|
Whiteboard: [fxcm-addr-telemetry]
Updated•1 year ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•