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

Update troubleshooting-web-deploy-problems-with-visual-studio.md #490

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

Conversation

chintalpatel89
Copy link

If you can add the changes I made than it will be very helpful for someone who might have missed this small setting under NSG/Firewall in the Azure. I wasted around 1hr to figure this out. And when I added I was successfully able to connect to Server through Web deploy from Visual Studio.

If you can add the changes I made than it will be very helpful for someone who might have missed this small setting under NSG/Firewall in the Azure. I wasted around 1hr to figure this out. And when I added I was successfully able to connect to Server through Web deploy from Visual Studio.
@chintalpatel89
Copy link
Author

Merging required

@Rick-Anderson
Copy link
Contributor

@shirhatti this looks worth adding. I can edit if you approve.

Make sure one has added inbout rule at NSG/Firewall level to allow request for port 8172 as shown below at all the levels i.e VNET/Subnets/NIC

Priority Name Port Protocol Source Destination Action
100 Port_8172 Any Any Any Any Allow
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can this be formatted as a Markdown table?

Comment on lines +54 to +55
Priority Name Port Protocol Source Destination Action
100 Port_8172 Any Any Any Any Allow
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Priority Name Port Protocol Source Destination Action
100 Port_8172 Any Any Any Any Allow

Copy link
Contributor

@Rick-Anderson Rick-Anderson May 28, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chintalpatel89 can you review and accept this suggestion? I would have commit it but you must have overridden the default to allow admins to push commits.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chintalpatel89 also address merge conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants