Improve security rule notification message

Bug #1430005 reported by Kahou Lei
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Low
Yosef Hoffman

Bug Description

If we create a rule that allow all ports to go thru, we are displaying the following notification message (see attached image too):

Successfully added rule:
-1:-1/icmp from 0.0.0.0/0

where -1:-1 doesn't deliver useful information.

Suggest to change -1:-1 to "any port" instead.

Revision history for this message
Kahou Lei (kahou82) wrote :
Changed in horizon:
assignee: nobody → Kahou Lei (kahou82)
Kahou Lei (kahou82)
Changed in horizon:
status: New → In Progress
Revision history for this message
Rob Cresswell (robcresswell-deactivatedaccount) wrote :

I can't recreate this in master; could you provide more information?

tags: added: low-hanging-fruit
Changed in horizon:
status: In Progress → Incomplete
Revision history for this message
BassT (mt104b) wrote :

Tested out all the different rules that applies to all ports on master. The message mentioned above has changed. The closest to it is the message when creating a rule for all ICMP where the message is

Successfully added rule:
-1:-1/icmp from 0.0.0.0/0

Changed in horizon:
status: Incomplete → Confirmed
importance: Undecided → Low
milestone: none → next
assignee: Kahou Lei (kahou82) → nobody
description: updated
Changed in horizon:
assignee: nobody → Yosef Hoffman (yh128t)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/313086

Changed in horizon:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/313610

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by Yosef Hoffman (<email address hidden>) on branch: master
Review: https://review.openstack.org/313610
Reason: Duplicate of https://review.openstack.org/#/c/313086/

Changed in horizon:
milestone: next → newton-3
Changed in horizon:
milestone: newton-3 → next
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/313086
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=9a535a1f848c7813169fdd7d042a29678c554092
Submitter: Jenkins
Branch: master

commit 9a535a1f848c7813169fdd7d042a29678c554092
Author: Yosef Hoffman <email address hidden>
Date: Thu May 5 11:12:23 2016 -0400

    Improve security rule notification message

    If from_port and to_port are both "-1" then return "any port"
    instead of "-1:-1". For example, instead of
     ``ALLOW -1:-1/icmp from 0.0.0.0/0``
    return
     ``ALLOW any port/icmp from 0.0.0.0/0``

    Change-Id: I0ded50a40089406fd69c8fa869839e46eecef352
    Closes-Bug: #1430005

Changed in horizon:
status: In Progress → Fix Released
Changed in horizon:
milestone: next → newton-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/horizon 10.0.0.0rc1

This issue was fixed in the openstack/horizon 10.0.0.0rc1 release candidate.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.