Security/Tracking protection/Bug triage: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
(Mention overall goals and add an "initial triage section.)
(Link to all of the metabugs.)
Line 13: Line 13:
#* Server-side bugs should be moved into '''Cloud Services::Server: Shavar'''
#* Server-side bugs should be moved into '''Cloud Services::Server: Shavar'''
# Breakage bugs should block one of these meta bugs:
# Breakage bugs should block one of these meta bugs:
#* <tt>tppaywall</tt> if TP is triggering a paywall / adblocker detection script (e.g. [https://github.com/sitexw/FuckAdBlock FuckAdBlock])
#* <tt>[https://bugzilla.mozilla.org/showdependencytree.cgi?id=1400031&hide_resolved=1 tppaywall]</tt> if TP is triggering a paywall / adblocker detection script (e.g. [https://github.com/sitexw/FuckAdBlock FuckAdBlock])
#* <tt>tpvideo</tt> if TP is blocking the playing of videos
#* <tt>[https://bugzilla.mozilla.org/showdependencytree.cgi?id=1400025&hide_resolved=1 tpvideo]</tt> if TP is blocking the playing of videos
#* <tt>tplogin</tt> if TP is breaking the login/registration flow of a site (including SSO providers)
#* <tt>[https://bugzilla.mozilla.org/showdependencytree.cgi?id=1470298&hide_resolved=1 tplogin]</tt> if TP is breaking the login/registration flow of a site (including SSO providers)
#* <tt>tpimages</tt> if TP is blocking necessary/useful images (excluding display ads)
#* <tt>[https://bugzilla.mozilla.org/showdependencytree.cgi?id=1470301&hide_resolved=1 tpimages]</tt> if TP is blocking necessary/useful images (excluding display ads)
#* <tt>trackingprotection</tt> otherwise
#* <tt>[https://bugzilla.mozilla.org/showdependencytree.cgi?id=1101005&hide_resolved=1 trackingprotection]</tt> otherwise
# Close paywall-related bugs as <tt>WONTFIX</tt>.
# Close paywall-related bugs as <tt>WONTFIX</tt>.
# Add the '''tp-entitylist''' whiteboard tag if the breakage could be mitigated by expanding the entity whitelist to linked domains owned by the same organization.
# Add the '''tp-entitylist''' whiteboard tag if the breakage could be mitigated by expanding the entity whitelist to linked domains owned by the same organization.

Revision as of 20:25, 27 June 2018

Once the bugs are fully triaged, there should be:

  • a priority on each bug
  • nothing with tp-needsrepro
  • no webcompat bugs linked to the trackingprotection meta bug

Initial triage

As bugs come into the Firefox::Tracking Protection component, the bug triage owner should follow these steps:

  1. Move misplaced bugs into the right component:
    • Platform bugs should be moved into Toolkit::Safe Browsing
    • Server-side bugs should be moved into Cloud Services::Server: Shavar
  2. Breakage bugs should block one of these meta bugs:
    • tppaywall if TP is triggering a paywall / adblocker detection script (e.g. FuckAdBlock)
    • tpvideo if TP is blocking the playing of videos
    • tplogin if TP is breaking the login/registration flow of a site (including SSO providers)
    • tpimages if TP is blocking necessary/useful images (excluding display ads)
    • trackingprotection otherwise
  3. Close paywall-related bugs as WONTFIX.
  4. Add the tp-entitylist whiteboard tag if the breakage could be mitigated by expanding the entity whitelist to linked domains owned by the same organization.
  5. Add the tp-needsrepro whiteboard tag if the problematic resource is not mentioned in the bug (see next section).
  6. For feature requests and bugs that require a product decision:
    1. Tag with the tp-product whiteboard tag.
    2. NEEDINFO :pdol with a suggested decision (typically P3 or WONTFIX).
  7. Set a priority on the bug:
    • tp-needsrepro: P3
    • tp-product: --
    • tp-entitylist: P2

Reproducing bugs

Someone looking at the tp-needsrepro bugs should:

  1. Ensure the URL of the broken page is in the URL field
  2. Close any unreproducible bugs with WORKSFORME.
  3. Mark bugs that have the same root cause (e.g. Facebook videos are blocked on site X) as DUPLICATE of one another.
  4. Find the blocked resource(s) responsible for the breakage (using uMatrix) and add it to the user story field.
  5. Tag bugs with the category that causes the breakage in the whiteboard field (use this mapping for Twitter, Google and Facebook resources in the Disconnect category):
    • tp-ads for the Advertising category
    • tp-analytics for the Analytics category
    • tp-social for the Social category
    • tp-content for the Content category (only part of the strict list)
  6. Remove the tp-needsrepro tag.
  7. Move the bug to Tech Evangelism: Desktop and reset the priority to --.