| bad1c10ffd2d4ac14f7bd9f4ef6a8982e711f566 |
|
19-Sep-2015 |
mukesh agrawal <quiche@google.com> |
shill: remove references to GLib
There are a number of references to GLib (the object),
even though no one uses GLib in a substantive way.
Remove these obsolete references.
Bug: 23386647
TEST=unit tests
Change-Id: I8d574980bc8e853c8b87ef64b581d64a71491201
/system/connectivity/shill/vpn/third_party_vpn_driver_unittest.cc
|
| c0beca55d290fe0b1c96d78cbbcf94b05c23f5a5 |
|
03-Sep-2015 |
Peter Qiu <zqiu@google.com> |
License shill to Apache2
This patch automatically replaced the license on all text files from
Chromium OS (BSD style) to AOSP (Apache2), keeping the original year as
a reference.
The license header was added to .gyp and .gypi, the NOTICE was added with a copy
of the Apache2 license and MODULE_LICENSE_* file was added.
BUG=23587594
TEST=grep 'Chromium OS Authors' doesn't find anything.
Change-Id: If41ede586c2465f5831fb7fee270ff41dbfdb596
/system/connectivity/shill/vpn/third_party_vpn_driver_unittest.cc
|
| a63da1dc556b1ec9a5b3676a75665ad1a5fda0bc |
|
26-Aug-2015 |
Bin Jin <binjin@chromium.org> |
shill: fix broken 3rd party vpn connection state change
CL:295388 replaced all "connected" connection state to "online",
but this change is not reflected in ThirdPartyVpnDriver and actually
caused failure on notifyConnectionStateChanged(connected) call.
Fix by ignoring Service::kStateOnline in ThirdPartyVpnDriver.
Also sync changes by the previous CL to cloned class in dbus/.
BUG=chromium:524214
TEST=manually tested
Change-Id: Ic082ea63c47ad449e16fc75fc1c2317c5b042dc3
Reviewed-on: https://chromium-review.googlesource.com/295081
Commit-Ready: Bin Jin <binjin@chromium.org>
Tested-by: Bin Jin <binjin@chromium.org>
Reviewed-by: Bartosz Fabianowski <bartfab@chromium.org>
Reviewed-by: Bin Jin <binjin@chromium.org>
/system/connectivity/shill/vpn/third_party_vpn_driver_unittest.cc
|
| e42d186ab3383b4a6bb267f490770af5932588b8 |
|
16-Jun-2015 |
Paul Stewart <pstew@chromium.org> |
shill: vpn: Switch pointer/ref spacing
Switch to Chrome style spacing.
BUG=chromium:501026
TEST=Unit tests
Change-Id: I7e196947d4d006a5ecc33d1a6bf03e0f4d4b1aa4
Reviewed-on: https://chromium-review.googlesource.com/278080
Reviewed-by: Rebecca Silberstein <silberst@chromium.org>
Commit-Queue: Paul Stewart <pstew@chromium.org>
Tested-by: Paul Stewart <pstew@chromium.org>
/system/connectivity/shill/vpn/third_party_vpn_driver_unittest.cc
|
| 8ac8272bd94dd0e58bb628fdee303d1fb7e42247 |
|
02-Mar-2015 |
Prabhu Kaliamoorthi <kaliamoorthi@chromium.org> |
shill: Fix error on UpdateConnectionState(Service::kStateFailure)
When an extension calls notifyConnectionStateChanged extension API
during a VPN failure, third party vpn driver is not cleaned up
properly, resulting in inconsistent state. This CL makes sure that
the cleanup happens correctly.
BUG=chromium:462235
TEST=Unit test updated and run, function verified manually.
Change-Id: I7cd2d4edbfa2338af31ff4862a8255c99a04744d
Reviewed-on: https://chromium-review.googlesource.com/255062
Reviewed-by: Paul Stewart <pstew@chromium.org>
Commit-Queue: Prabhu Kaliamoorthi <kaliamoorthi@chromium.org>
Tested-by: Prabhu Kaliamoorthi <kaliamoorthi@chromium.org>
/system/connectivity/shill/vpn/third_party_vpn_driver_unittest.cc
|
| 1f58903c02e9a8e29816f7a7659afa331b005d9c |
|
23-Feb-2015 |
Prabhu Kaliamoorthi <kaliamoorthi@chromium.org> |
shill: Cleanup and fix problem with default address in vpn
When default address is set as the first value in excluded list
passed to SetParameters DBus API of ThirdPartyVpn driver, the
routing entries are not correctly setup since the route for default
address is found to be local. This CL ensures that this problem
does not happen by swapping the address when the first entry is
default. The CL further cleans up IPConfig.
BUG=chromium:459609
TEST=Unit tests and manual testing of the feature.
Change-Id: I0fa167aa6cb09e467a33f876dd98ef5a9000fbb4
Reviewed-on: https://chromium-review.googlesource.com/251990
Reviewed-by: Paul Stewart <pstew@chromium.org>
Tested-by: Prabhu Kaliamoorthi <kaliamoorthi@chromium.org>
Commit-Queue: Prabhu Kaliamoorthi <kaliamoorthi@chromium.org>
/system/connectivity/shill/vpn/third_party_vpn_driver_unittest.cc
|
| e0e205bf5e625398d02be89bda076ea9fd761896 |
|
20-Feb-2015 |
Prabhu Kaliamoorthi <kaliamoorthi@chromium.org> |
shill: Cleanup third party vpn driver in shill
This CL performs the following changes.
1) Depricated parameters such as bypass_ip_from_tunnel
are removed.
2) Inclusion lists and exclusion lists are used instead
and are made mandatory settings.
3) Hard coded strings in parameters are replaced with
constants.
4) Default route setting is removed for third party vpn.
The extension is now explicitly required to set the
tunnel interface as default.
5) Iptables setting made using permisison broker are
explicitly torn down instead of relying on the lifeline.
BUG=chromium:459609
TEST=Manual testing of the feature.
Change-Id: Ibfca4a7485bfbe8e20d68f7159b18f8a38b0c608
Reviewed-on: https://chromium-review.googlesource.com/251550
Reviewed-by: Paul Stewart <pstew@chromium.org>
Tested-by: Prabhu Kaliamoorthi <kaliamoorthi@chromium.org>
Commit-Queue: Prabhu Kaliamoorthi <kaliamoorthi@chromium.org>
/system/connectivity/shill/vpn/third_party_vpn_driver_unittest.cc
|
| 762bfb8ab200a387fe732ec92423a5f0afe11bcf |
|
06-Feb-2015 |
Prabhu Kaliamoorthi <kaliamoorthi@chromium.org> |
shill: Add support in shill to route selected traffic to ThirdPartyVpn
This CL adds capability to shill deamon to route selected network
traffic to ThirdPartyVpn by using multiple routing tables.
It further enables a list of IP blocks in CIDR notation to be excluded
from the VPN traffic rather than a single trusted_ip. The existing
trusted_ip mechanism to do this is removed to avoid redundant approaches
to solve the same problem.
Third party VPN driver is modified to make use of the new interfaces to
exclude and include a list of IP blocks in CIDR format from the VPN
traffic.
BUG=chromium:407541
TEST=Unittests and manual testing
Change-Id: Id185d3e9bf91d018b55899adaa562d3192d261cf
Reviewed-on: https://chromium-review.googlesource.com/247030
Commit-Queue: Prabhu Kaliamoorthi <kaliamoorthi@chromium.org>
Reviewed-by: Paul Stewart <pstew@chromium.org>
Tested-by: Prabhu Kaliamoorthi <kaliamoorthi@chromium.org>
/system/connectivity/shill/vpn/third_party_vpn_driver_unittest.cc
|
| 2240e8c03451c6b6f21eb8944d8a1c0747ac10b3 |
|
06-Nov-2014 |
Ben Chan <benchan@chromium.org> |
shill: vpn: Move vpn code to its own sub-directory.
This CL is part of a series of CLs, which aim to reorganzie shill's flat
source code structure into a more modular form by moving technology
specific code into their own sub-directories.
BUG=chromium:433419
TEST=`USE='vpn' FEATURES=test emerge-$BOARD shill`
TEST=`USE='vpn clang asan' FEATURES=test emerge-$BOARD shill`
Change-Id: I02d23511a80655098d50e248321c77d23f8caaa9
Reviewed-on: https://chromium-review.googlesource.com/229870
Reviewed-by: Paul Stewart <pstew@chromium.org>
Commit-Queue: Ben Chan <benchan@chromium.org>
Tested-by: Ben Chan <benchan@chromium.org>
/system/connectivity/shill/vpn/third_party_vpn_driver_unittest.cc
|