MicrosoftDocs / azure-docs

Open source documentation of Microsoft Azure

https://docs.microsoft.com/azure

Creative Commons Attribution 4.0 International

9.98k stars 20.95k forks source link

Update synapse-workspace-ip-firewall.md ports and protocols #121980

Closed alrosado closed 1 week ago

alrosado commented 1 week ago

There is mention of a required port of 1443 for Synapse Studio. I'm assuming that this is a typo and the port is actually 1433. Requesting an update to the document to reflect the correct port.

prmerger-automator[bot] commented 1 week ago

@alrosado : Thanks for your contribution! The author(s) have been notified to review your proposed change. @ashinMSFT

learn-build-service-prod[bot] commented 1 week ago

Learn Build status updates of commit b8a47ae:

:white_check_mark: Validation status: passed

File	Status	Preview URL	Details
articles/synapse-analytics/security/synapse-workspace-ip-firewall.md	:white_check_mark:Succeeded

For more details, please refer to the build report.

For any questions, please:

Try searching the learn.microsoft.com contributor guides
Post your question in the Learn support channel

Jak-MS commented 1 week ago

@Danzhang-msft

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

@Danzhang-msft - can you update the author metadata value; it's not a valid GitHub alias.

label:"aq-pr-triaged"

@MicrosoftDocs/public-repo-pr-review-team

Danzhang-msft commented 1 week ago

Our doc is correct. It is 1443 not 1433 for Synapse studio.

Court72 commented 1 week ago

Closing per https://github.com/MicrosoftDocs/azure-docs/pull/121980#issuecomment-2080313465. Thanks!