Add XMPP logging to diagnose connection failures |
|||||||
Issue descriptionWe are seeing an incidence of XMPP timeouts higher than the historical average. We should add logging to diagnose where the failure occurs.
,
Oct 18 2017
,
Oct 19 2017
,
Oct 19 2017
,
Oct 19 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/26131f43639db30d7e5cdb395819a70242140ade commit 26131f43639db30d7e5cdb395819a70242140ade Author: Jamie Walch <jamiewalch@chromium.org> Date: Thu Oct 19 18:31:15 2017 Add XMPP logging. Bug: 775748 Change-Id: Id49b1a54fbcbb4ead1b093822be59753420ef983 Reviewed-on: https://chromium-review.googlesource.com/724293 Commit-Queue: Jamie Walch <jamiewalch@chromium.org> Reviewed-by: Kelvin Poon <kelvinp@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#509846}(cherry picked from commit 48d251f9bfe6b49da2b47e63cebf088bac364391) Reviewed-on: https://chromium-review.googlesource.com/728403 Reviewed-by: Jamie Walch <jamiewalch@chromium.org> Cr-Commit-Position: refs/branch-heads/3239@{#80} Cr-Branched-From: adb61db19020ed8ecee5e91b1a0ea4c924ae2988-refs/heads/master@{#508578} [modify] https://crrev.com/26131f43639db30d7e5cdb395819a70242140ade/remoting/webapp/base/js/xmpp_connection.js [modify] https://crrev.com/26131f43639db30d7e5cdb395819a70242140ade/remoting/webapp/base/js/xmpp_login_handler.js
,
Oct 19 2017
,
Oct 19 2017
This got already merged at #5. Hence, removing "Merge-Request-63" label. jamiewalch@, is the change look good in Canary?
,
Oct 19 2017
It's a Chromoting change, so it won't appear in Chrome canary. |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by bugdroid1@chromium.org
, Oct 18 2017