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 Cant create ICMP firewall rules on windows #174

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

Conversation

alexandrezia
Copy link

Description

on windows icmp rules must not specify port

Issues Resolved

#156

Check List

@ianau74
Copy link

ianau74 commented Oct 27, 2017

Can you fix the 2 failing checks so this can get merged into master? They seem like pretty trivial fixes.

@mjavier2k
Copy link

Im having the same issue and a merge into master will be appreciated.

@smastrorocco
Copy link

Any chance we can get this merged?

@RiverHeart
Copy link

Hey @alexandrezia I realize this was opened a long time ago but would you be willing to resubmit this? If not, would it be okay if I resubmitted this as a different PR attribution to you?

@alexandre-zia-ifood
Copy link

Hey @alexandrezia I realize this was opened a long time ago but would you be willing to resubmit this? If not, would it be okay if I resubmitted this as a different PR attribution to you?

Hey, it was too long ago, I don't have context about this anymore, please go ahead, no worries.

@RiverHeart
Copy link

Hey @alexandrezia I realize this was opened a long time ago but would you be willing to resubmit this? If not, would it be okay if I resubmitted this as a different PR attribution to you?

Hey, it was too long ago, I don't have context about this anymore, please go ahead, no worries.

Thanks @alexandrezia :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Medium Will bring visible benefit to the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants