Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fix] Remove port when both connection will be closed #524

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

eldertek
Copy link

Hello, in the past when closing e.g : "port 8080 - tcp - ipv4" when "port - tcp - 8080 - ipv6" is already close, the port stay in firewall, this pr will allows the port to be removed from entry of the firewall webadmin tool.

@eldertek
Copy link
Author

Sorry did a bad explanation
when port 1000 is opened in tcp and closed in udp, and when user closes the tcp, port 1000 remain in webadmin with the fields udp and tcp grayed out

my pr allows to completely close the port when ether udp or tcp is already closed

@eldertek

This comment was marked as outdated.

@eldertek

This comment was marked as outdated.

@eldertek
Copy link
Author

eldertek commented Sep 1, 2023

@alexAubin

@alexAubin
Copy link
Member

Yes, we've seen your PR

  • I'm not the only maintainer in YunoHost, no need to ping me personally
  • There's always literally thousands of tasks in YunoHost
  • People tend to have a life beside YunoHost

Your PR will be merged when somebody will have the time and energy to properly review it

@eldertek eldertek closed this Oct 11, 2023
@zamentur zamentur reopened this Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants