azure-docs icon indicating copy to clipboard operation
azure-docs copied to clipboard

ADF Doesnt Support disable public access for UI

Open michalmar opened this issue 2 years ago • 1 comments

As per article https://learn.microsoft.com/en-us/answers/questions/154590/how-to-disable-public-access-to-adf-portal.html and empirical tests, Azure Data Factory cannot be disabled public access to authoring UI. Even if we follow steps in this article, the UI is still accessible and we can see all resource (or data).

Can you share if you plan something link Private Endpoint to ADF UI?


Document Details

Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

michalmar avatar Dec 02 '22 10:12 michalmar

@michalmar Thanks for your feedback! We will investigate and update as appropriate.

Naveenommi-MSFT avatar Dec 02 '22 12:12 Naveenommi-MSFT

@michalmar

Thank you for bringing this to our attention. I've assigned this issue to the author who will investigate and update as appropriate.

@lrtoyou1223

could you please review this and update as appropriate.

Any updates on this?

Rajini09 avatar Dec 20 '22 20:12 Rajini09

The documentation mentions - If you want to use the private endpoint for authoring and monitoring the data factory in your virtual network, select portal as Target sub-resource. But it isn't restricting the access to portal/adf studio only through virtual networks. After creating that Target sub-resource authors can still access via public network. Can we please explicitly add a line saying this Target sub-resource doesnt restrict access to the data factory studio over public network. image

subashsivaji avatar Dec 22 '22 14:12 subashsivaji

Only block public access to UI is not a feasible solution since people still can access ADF service through API. So the better solution is to disable all ARM level access. Please refer to https://learn.microsoft.com/en-us/azure/azure-resource-manager/management/create-private-link-access-portal?source=docs

lrtoyou1223 avatar Mar 03 '23 01:03 lrtoyou1223

Thanks for providing feedback that helps improve our documentation. This issue has been resolved and we are closing the issue. #please-close

lrtoyou1223 avatar Mar 03 '23 01:03 lrtoyou1223

Only block public access to UI is not a feasible solution since people still can access ADF service through API. So the better solution is to disable all ARM level access. Please refer to https://learn.microsoft.com/en-us/azure/azure-resource-manager/management/create-private-link-access-portal?source=docs

Do you mean RBAC with this comment? I'm not sure how private link is going to close access to data factory resource via management.azure.com and *.svc.datafactory.azure.com (and adf.azure.com).

This limitation was clearly mentioned by @subashsivaji

But it isn't restricting the access to portal/adf studio only through virtual networks. After creating that Target sub-resource authors can still access via public network.

The documetation itself or in the security baseline documentation mentions above-mentioned limitation: https://learn.microsoft.com/en-us/security/benchmark/azure/baselines/data-factory-security-baseline.


Security baseline mentions following for data factory: image

Following use case is issue for some organizations:

  • open authoring portal using public internet
  • open dataset
  • hit preview
  • data will flow through public internet

I'm not sure if it is possible to configure data factory as security baseline or linked documentation describe. Mentioning about limitations on github issue or QnA site without documentation updates is IMHO kind of bad PR/DX.

jikuja avatar Mar 10 '23 17:03 jikuja

@lrtoyou1223 We have enabled the private dns zone privatelink.adf.azure.com but now https://adf.azure.com has become unreachable. The temporary workaround we applied is to add a CNAME from the public dns record to the private dns zone: portal CNAME 3600 datafactoryv2.trafficmanager.net

Can you shed some light on this? Could you add the recommended configuration to the docs?

cveld avatar Apr 14 '23 09:04 cveld