xref: /aosp_15_r20/external/cronet/net/docs/bug-triage-labels.md (revision 6777b5387eb2ff775bb5750e3f5d96f37fb7352b)
1*6777b538SAndroid Build Coastguard Worker# Chrome Network Bug Triage : Components and labels
2*6777b538SAndroid Build Coastguard Worker
3*6777b538SAndroid Build Coastguard Worker## Some network component caveats
4*6777b538SAndroid Build Coastguard Worker
5*6777b538SAndroid Build Coastguard Worker* **Internals>Network>SSL**
6*6777b538SAndroid Build Coastguard Worker
7*6777b538SAndroid Build Coastguard Worker    This includes issues that should be also tagged as **Security>UX**
8*6777b538SAndroid Build Coastguard Worker    (certificate error pages or other security interstitials, omnibox indicators
9*6777b538SAndroid Build Coastguard Worker    that a page is secure), and more general SSL issues.  If you see requests
10*6777b538SAndroid Build Coastguard Worker    that die in the SSL negotiation phase, in particular, this is often the
11*6777b538SAndroid Build Coastguard Worker    correct component.
12*6777b538SAndroid Build Coastguard Worker
13*6777b538SAndroid Build Coastguard Worker* **Internals>Network>Cache**
14*6777b538SAndroid Build Coastguard Worker
15*6777b538SAndroid Build Coastguard Worker    The cache is the layer that handles most range request logic (Though range
16*6777b538SAndroid Build Coastguard Worker    requests may also be issued by the PDF plugin, XHRs, or other components).
17*6777b538SAndroid Build Coastguard Worker
18*6777b538SAndroid Build Coastguard Worker* **Internals>Network>HTTP**
19*6777b538SAndroid Build Coastguard Worker
20*6777b538SAndroid Build Coastguard Worker    Typically not used.  Unclear what it covers, and there's no specific HTTP
21*6777b538SAndroid Build Coastguard Worker    owner.
22*6777b538SAndroid Build Coastguard Worker
23*6777b538SAndroid Build Coastguard Worker* **Internals>Network>Logging**
24*6777b538SAndroid Build Coastguard Worker
25*6777b538SAndroid Build Coastguard Worker    Covers **about:net-internals**, **about:net-export** as well as the what's
26*6777b538SAndroid Build Coastguard Worker    sent to the NetLog.
27*6777b538SAndroid Build Coastguard Worker
28*6777b538SAndroid Build Coastguard Worker* **Internals>Network>Connectivity**
29*6777b538SAndroid Build Coastguard Worker
30*6777b538SAndroid Build Coastguard Worker    Issues related to switching between networks, `ERR_NETWORK_CHANGED`, Chrome
31*6777b538SAndroid Build Coastguard Worker    thinking it's online when it's not / navigator.onLine inaccuracies, etc.
32*6777b538SAndroid Build Coastguard Worker
33*6777b538SAndroid Build Coastguard Worker* **Internals>Network>Filters**
34*6777b538SAndroid Build Coastguard Worker
35*6777b538SAndroid Build Coastguard Worker    Covers gzip, deflate and brotli issues.  `ERR_CONTENT_DECODING_FAILED`
36*6777b538SAndroid Build Coastguard Worker    indicates a problem at this layer, and bugs here can also cause response
37*6777b538SAndroid Build Coastguard Worker    body corruption.
38*6777b538SAndroid Build Coastguard Worker
39*6777b538SAndroid Build Coastguard Worker## Common non-network components
40*6777b538SAndroid Build Coastguard Worker
41*6777b538SAndroid Build Coastguard WorkerBugs in these areas often receive the **Internals>Network** component, though
42*6777b538SAndroid Build Coastguard Workerthey fall largely outside the purview of the network stack team:
43*6777b538SAndroid Build Coastguard Worker
44*6777b538SAndroid Build Coastguard Worker* **UI>Browser>Downloads**
45*6777b538SAndroid Build Coastguard Worker
46*6777b538SAndroid Build Coastguard Worker    Despite the name, this covers all issues related to downloading a file
47*6777b538SAndroid Build Coastguard Worker    except saving entire pages (which is **Blink>SavePage**), not just UI
48*6777b538SAndroid Build Coastguard Worker    issues.  Most downloads bugs will have the word "download" or "save as" in
49*6777b538SAndroid Build Coastguard Worker    the description.  Issues with the HTTP server for the Chrome binaries are
50*6777b538SAndroid Build Coastguard Worker    not downloads bugs.
51*6777b538SAndroid Build Coastguard Worker
52*6777b538SAndroid Build Coastguard Worker* **Services>Safebrowsing**
53*6777b538SAndroid Build Coastguard Worker
54*6777b538SAndroid Build Coastguard Worker    Bugs that have to do with the process by which a URL or file is determined
55*6777b538SAndroid Build Coastguard Worker    to be dangerous based on our databases, or the resulting interstitials.
56*6777b538SAndroid Build Coastguard Worker    Determination of danger based purely on content-type or file extension
57*6777b538SAndroid Build Coastguard Worker    belongs in **UI>Browser>Downloads**, not SafeBrowsing.
58*6777b538SAndroid Build Coastguard Worker
59*6777b538SAndroid Build Coastguard Worker* **Blink>Forms**
60*6777b538SAndroid Build Coastguard Worker
61*6777b538SAndroid Build Coastguard Worker    Issues submitting forms, forms having weird data, forms sending the wrong
62*6777b538SAndroid Build Coastguard Worker    method, etc.
63*6777b538SAndroid Build Coastguard Worker
64*6777b538SAndroid Build Coastguard Worker* **Blink>Loader**
65*6777b538SAndroid Build Coastguard Worker
66*6777b538SAndroid Build Coastguard Worker    Cross origin issues are sometimes loader related.  Blink also has an
67*6777b538SAndroid Build Coastguard Worker    in-memory cache, and when it's used, requests don't appear in
68*6777b538SAndroid Build Coastguard Worker    about:net-internals.  Requests for the same URL are also often merged there
69*6777b538SAndroid Build Coastguard Worker    as well.  This does *not* cover issues with content/browser/loader/ files.
70*6777b538SAndroid Build Coastguard Worker
71*6777b538SAndroid Build Coastguard Worker* **Blink>ServiceWorker**
72*6777b538SAndroid Build Coastguard Worker
73*6777b538SAndroid Build Coastguard Worker* **Blink>Network>WebSockets**
74*6777b538SAndroid Build Coastguard Worker
75*6777b538SAndroid Build Coastguard Worker    Issues with the WebSockets.  Attach this component to any issue about the
76*6777b538SAndroid Build Coastguard Worker    WebSocket feature regardless of where the cause of the issue is (net/ or
77*6777b538SAndroid Build Coastguard Worker    Blink).
78*6777b538SAndroid Build Coastguard Worker
79*6777b538SAndroid Build Coastguard Worker* **Blink>Network>FetchAPI**
80*6777b538SAndroid Build Coastguard Worker
81*6777b538SAndroid Build Coastguard Worker    Generic issues with the Fetch API - missing request or response headers,
82*6777b538SAndroid Build Coastguard Worker    multiple headers, etc.  These will often run into issues in certain corner
83*6777b538SAndroid Build Coastguard Worker    cases (Cross origin / CORS, proxy, whatever).  Attach all components that
84*6777b538SAndroid Build Coastguard Worker    seem appropriate.
85*6777b538SAndroid Build Coastguard Worker
86*6777b538SAndroid Build Coastguard Worker* **Blink>Network>XHR**
87*6777b538SAndroid Build Coastguard Worker
88*6777b538SAndroid Build Coastguard Worker    Generic issues with sync/async XHR requests.
89*6777b538SAndroid Build Coastguard Worker
90*6777b538SAndroid Build Coastguard Worker* **Blink>WebRTC>Network**
91*6777b538SAndroid Build Coastguard Worker
92*6777b538SAndroid Build Coastguard Worker    Anything WebRTC-related does not use the net stack and should go here.
93*6777b538SAndroid Build Coastguard Worker
94*6777b538SAndroid Build Coastguard Worker* **Services>Sync**
95*6777b538SAndroid Build Coastguard Worker
96*6777b538SAndroid Build Coastguard Worker    Sharing data/tabs/history/passwords/etc between machines not working.
97*6777b538SAndroid Build Coastguard Worker
98*6777b538SAndroid Build Coastguard Worker* **Services>Chromoting**
99*6777b538SAndroid Build Coastguard Worker
100*6777b538SAndroid Build Coastguard Worker* **Platform>Extensions**
101*6777b538SAndroid Build Coastguard Worker
102*6777b538SAndroid Build Coastguard Worker    Issues extensions loading / not loading / hanging.
103*6777b538SAndroid Build Coastguard Worker
104*6777b538SAndroid Build Coastguard Worker* **Platform>Extensions>API**
105*6777b538SAndroid Build Coastguard Worker
106*6777b538SAndroid Build Coastguard Worker    Issues with network related extension APIs should have this component.
107*6777b538SAndroid Build Coastguard Worker    chrome.webRequest is the big one, I believe, but there are others.
108*6777b538SAndroid Build Coastguard Worker
109*6777b538SAndroid Build Coastguard Worker* **Internals>Plugins>Pepper[>SDK]**
110*6777b538SAndroid Build Coastguard Worker
111*6777b538SAndroid Build Coastguard Worker* **UI>Browser>Omnibox**
112*6777b538SAndroid Build Coastguard Worker
113*6777b538SAndroid Build Coastguard Worker    Basically any issue with the omnibox.  URLs being treated as search queries
114*6777b538SAndroid Build Coastguard Worker    rather than navigations, dropdown results being weird, not handling certain
115*6777b538SAndroid Build Coastguard Worker    Unicode characters, etc.  If the issue is new TLDs not being recognized by
116*6777b538SAndroid Build Coastguard Worker    the omnibox, that's due to Chrome's TLD list being out of date, and not an
117*6777b538SAndroid Build Coastguard Worker    omnibox issue.  Such TLD issues should be duped against
118*6777b538SAndroid Build Coastguard Worker    http://crbug.com/37436.
119*6777b538SAndroid Build Coastguard Worker
120*6777b538SAndroid Build Coastguard Worker* **Internals>Media>Network**
121*6777b538SAndroid Build Coastguard Worker
122*6777b538SAndroid Build Coastguard Worker    Issues related to media.  These often run into the 6 requests per hostname
123*6777b538SAndroid Build Coastguard Worker    issue, and also have fun interactions with the cache, particularly in the
124*6777b538SAndroid Build Coastguard Worker    range request case.
125*6777b538SAndroid Build Coastguard Worker
126*6777b538SAndroid Build Coastguard Worker* **Internals>Plugins>PDF**
127*6777b538SAndroid Build Coastguard Worker
128*6777b538SAndroid Build Coastguard Worker    Issues loading PDF files.  These are often related to range requests, which
129*6777b538SAndroid Build Coastguard Worker    also have some logic at the Internals>Network>Cache layer.
130*6777b538SAndroid Build Coastguard Worker
131*6777b538SAndroid Build Coastguard Worker* **UI>Browser>Navigation**
132*6777b538SAndroid Build Coastguard Worker
133*6777b538SAndroid Build Coastguard Worker    Despite the name, this covers all issues related to page navigation, not
134*6777b538SAndroid Build Coastguard Worker    just UI issues.
135*6777b538SAndroid Build Coastguard Worker
136*6777b538SAndroid Build Coastguard Worker* **UI>Browser>History**
137*6777b538SAndroid Build Coastguard Worker
138*6777b538SAndroid Build Coastguard Worker    Issues which only appear with forward/back navigation.
139*6777b538SAndroid Build Coastguard Worker
140*6777b538SAndroid Build Coastguard Worker* **OS>Systems>Network** / **OS>Systems>Mobile** / **OS>Systems>Bluetooth**
141*6777b538SAndroid Build Coastguard Worker
142*6777b538SAndroid Build Coastguard Worker    These should be used for issues with Chrome OS's platform network code, and
143*6777b538SAndroid Build Coastguard Worker    not net/ issues on Chrome OS.
144*6777b538SAndroid Build Coastguard Worker
145*6777b538SAndroid Build Coastguard Worker* **Blink>SecurityFeature**
146*6777b538SAndroid Build Coastguard Worker
147*6777b538SAndroid Build Coastguard Worker    CORS / Cross origin issues.  Main frame cross-origin navigation issues are
148*6777b538SAndroid Build Coastguard Worker    often actually **UI>Browser>Navigation** issues.
149*6777b538SAndroid Build Coastguard Worker
150*6777b538SAndroid Build Coastguard Worker* **Privacy**
151*6777b538SAndroid Build Coastguard Worker
152*6777b538SAndroid Build Coastguard Worker    Privacy related bug (History, cookies discoverable by an entity that
153*6777b538SAndroid Build Coastguard Worker    shouldn't be able to do so, incognito state being saved in memory or on disk
154*6777b538SAndroid Build Coastguard Worker    beyond the lifetime of incognito tabs, etc).  Generally used in conjunction
155*6777b538SAndroid Build Coastguard Worker    with other components.
156*6777b538SAndroid Build Coastguard Worker
157*6777b538SAndroid Build Coastguard Worker## Common labels
158*6777b538SAndroid Build Coastguard Worker
159*6777b538SAndroid Build Coastguard Worker* **Type-Bug-Security**
160*6777b538SAndroid Build Coastguard Worker
161*6777b538SAndroid Build Coastguard Worker    Security related bug (Allows for code execution from remote site, allows
162*6777b538SAndroid Build Coastguard Worker    crossing security boundaries, unchecked array bounds, etc) should be tagged
163*6777b538SAndroid Build Coastguard Worker    with this label.
164