It2Me host shouldn't ignore port range policy |
||||||
Issue descriptionCurrently It2Me host ignores port-range policy. The policy is only applied for me2me host.
,
May 26 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/49ccc86fa6f0112e768dda3b7dd2d78529275ca8 commit 49ccc86fa6f0112e768dda3b7dd2d78529275ca8 Author: gusss <gusss@google.com> Date: Fri May 26 17:35:41 2017 Host port range policy is no longer ignored in it2me host. Adds UpdateHostUdpPortRangePolicy into It2MeHost, which is called whenever a policy update contains a port range setting. This port range setting is then stored, and is later used to set the appropriate fields in the NetworkSettings object passed into the TransportContext. BUG= 723093 Review-Url: https://codereview.chromium.org/2901033002 Cr-Commit-Position: refs/heads/master@{#475047} [modify] https://crrev.com/49ccc86fa6f0112e768dda3b7dd2d78529275ca8/remoting/host/chromoting_host.h [modify] https://crrev.com/49ccc86fa6f0112e768dda3b7dd2d78529275ca8/remoting/host/it2me/it2me_host.cc [modify] https://crrev.com/49ccc86fa6f0112e768dda3b7dd2d78529275ca8/remoting/host/it2me/it2me_host.h [modify] https://crrev.com/49ccc86fa6f0112e768dda3b7dd2d78529275ca8/remoting/host/it2me/it2me_host_unittest.cc
,
May 26 2017
,
May 30 2017
,
May 30 2017
Your change meets the bar and is auto-approved for M60. Please go ahead and merge the CL to branch 3112 manually. Please contact milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 30 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8ca472105b2cb318faaf61882000504fd8f7649f commit 8ca472105b2cb318faaf61882000504fd8f7649f Author: Sergey Ulanov <sergeyu@chromium.org> Date: Tue May 30 18:33:08 2017 Host port range policy is no longer ignored in it2me host. Adds UpdateHostUdpPortRangePolicy into It2MeHost, which is called whenever a policy update contains a port range setting. This port range setting is then stored, and is later used to set the appropriate fields in the NetworkSettings object passed into the TransportContext. BUG= 723093 Review-Url: https://codereview.chromium.org/2901033002 Cr-Original-Commit-Position: refs/heads/master@{#475047} Review-Url: https://codereview.chromium.org/2912143002 . Cr-Commit-Position: refs/branch-heads/3112@{#35} Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897} [modify] https://crrev.com/8ca472105b2cb318faaf61882000504fd8f7649f/remoting/host/chromoting_host.h [modify] https://crrev.com/8ca472105b2cb318faaf61882000504fd8f7649f/remoting/host/it2me/it2me_host.cc [modify] https://crrev.com/8ca472105b2cb318faaf61882000504fd8f7649f/remoting/host/it2me/it2me_host.h [modify] https://crrev.com/8ca472105b2cb318faaf61882000504fd8f7649f/remoting/host/it2me/it2me_host_unittest.cc
,
May 30 2017
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by jamiewa...@chromium.org
, May 23 2017Status: Assigned (was: Untriaged)