You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by re...@apache.org on 2015/09/27 14:11:28 UTC

[09/21] git commit: updated refs/heads/master to 3ded3e9

Merge pull request #851 from SudharmaJain/cs-8864

CLOUDSTACK-8864: Not able to add TCP port forwarding rule in VPN for specific ports

Setting port forwarding rules for port 500,1701 and 4500 after enabling VPN, gives the error message "The range specified, xxxx, conflicts with rule xxxx which has xxxx." This happens because the rules added for vpn doesn't have a matching condition to allow port forwarding rules.

Added a unit test to verify the detectRulesConflict function of FirewallManagerImpl.

* pr/851:
  CLOUDSTACK-8864: Not able to add TCP port forwarding rule in VPN for specific ports

Signed-off-by: Remi Bergsma <gi...@remi.nl>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/7d555542
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/7d555542
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/7d555542

Branch: refs/heads/master
Commit: 7d5555429b90fcb9e1456ea858d5163b41ee41ab
Parents: 8367911 96c38bf
Author: Remi Bergsma <gi...@remi.nl>
Authored: Thu Sep 24 15:35:16 2015 +0200
Committer: Remi Bergsma <gi...@remi.nl>
Committed: Thu Sep 24 15:35:16 2015 +0200

----------------------------------------------------------------------
 .../network/firewall/FirewallManagerImpl.java   |  3 +-
 .../network/firewall/FirewallManagerTest.java   | 83 +++++++++++++++++---
 2 files changed, 76 insertions(+), 10 deletions(-)
----------------------------------------------------------------------