ufw

Add IGMP protocols to cli tool

Bug #946322 reported by Leon
52
This bug affects 11 people
Affects Status Importance Assigned to Milestone
ufw
Fix Released
Wishlist
Unassigned

Bug Description

Would it be possible to add ICMP & IGMP to the protocols ufw can use? There are ways around it and it isn't needed often but it can be handy and it would be nice for completeness' sake.

Tags: wishlist
Changed in ufw:
importance: Undecided → Wishlist
status: New → Confirmed
summary: - Add ICMP/IGMP protocols
+ Add ICMP/IGMP protocols to cli tool
Revision history for this message
Pali (pali) wrote : Re: Add ICMP/IGMP protocols to cli tool

Allowing igmp packets is needed for configuring correct multicast support

Revision history for this message
Scott Bertilson (ssb-v) wrote :

This seems to be a trivial change at least as far as igmp is concerned:
--- /usr/lib/python3/dist-packages/ufw/common.py.orig 2014-02-28 15:50:59.000000000 -0600
+++ /usr/lib/python3/dist-packages/ufw/common.py 2015-01-13 11:38:45.249090960 -0600
@@ -246,6 +246,7 @@
            protocol == "ipv6" or \
            protocol == "esp" or \
            protocol == "ah" or \
+ protocol == "igmp" or \
            protocol == "any":
             self.protocol = protocol
         else:

This would be very helpful since it allows one to enable IPv4 multicast using the CLI alone.

I hesitate to suggest that icmp is as simple because there are built-in rules which may need to be taken into account.

no longer affects: ufw (Ubuntu)
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

It isn't quite as easy as comment #2-- for example, igmp should only be used with ipv4 addresses, you shouldn't specify a port and there needs to be test cases. However, I've added preliminary support for igmp just now. icmp is a whole other thing and may be addressed at a future date.

summary: - Add ICMP/IGMP protocols to cli tool
+ Add IGMP protocols to cli tool
Changed in ufw:
status: Confirmed → Fix Committed
milestone: none → 0.34
Changed in ufw:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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