Issue metadata
Sign in to add a comment
|
Regression: Unnecessarily continuous reloading of Network not available page is seen in sign-out screen when random values are entered in Custom Name servers |
||||||||||||||||||||
Issue descriptionChrome Version:67.0.3383.0/10538.0.0 dev channel Candy,Peppy,Celes OS:Chrome OS What steps will reproduce the problem? (1)In sign-out screen>> click on Ubertray, Network and then on connected WiFi (2)Change the Name servers to Custom Name servers and give some random values (3)Then click on Add person and observe Actual: Unnecessarily continuous reloading of Network not available page is seen Expected: No such issue should be seen This is a Regression issue as same is working fine in M62 NOTE: Issue is seen on M63, M64, M65 and M66 also @Stevenjb: Please confirm the issue
,
Apr 2 2018
Good
,
Apr 2 2018
The useful part of the repro starts at 0:38. The 'Network not available' screen flashes.
,
Apr 7 2018
+ Ben + Zach This is because error screen is shown on the webview error: "<webview>: The load has aborted with error -137: ERR_NAME_RESOLUTION_FAILED.", source: extensions::webViewEvents (283) and it's closed on shill reporting that network is connected. Gaia is reloaded, another ERR_NAME_RESOLUTION_FAILED happens, another error screen, stc... So we can either handle WebView errors in out code in a more granular way, or make shill use the given name servers to do a network test. It might also be useful to include a word about possible name server failure to the error message. Zach, Ben, WDYT?
,
Apr 9 2018
We definitely need to handle this somehow, more for the case of mistyped custom name servers than for random text entry. Agree that adding some warning messaging is good too.
,
Apr 9 2018
There is no simple way to 'make shill use the given name servers to do a network test'. This is something we need to handle correctly in the WebUI. It seems like a bad proxy configuration or a network portal could have as similar problem? In general we can not assume that a connected network equates to an available internet connection.
,
Apr 13 2018
We should probably also include error code/ID to the error screen.
,
Nov 21
,
Jan 15
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by kebalaji@chromium.org
, Apr 2 2018