Bad assumption in isCosmosEndpoint.ts makes it impossible to run azurite table service using custom ports.

[cognot]

• Package Name: @azure/data-tables
• Package Version: 13.3.0
• Operating system: Any
• [* ] nodejs
  • version: 18.X
• browser
  • name/version:
• typescript
  • version:
• Is the bug related to documentation in
  • README.md
  • source code documentation
  • SDK API docs on https://learn.microsoft.com

Describe the bug
Since 13.3.0 and the introduction of the CosmoDB table support, it is impossible to start the azurite emulator on custom ports for the table service and make table access with a SAS key anymore.

To Reproduce
Steps to reproduce the behavior:

1. Start Azurite table service on a custom port (for example 20002)
2. Instantiate a table client with a SAS key (note: in our case the SAS key was generated with a service that was still using @azure/data-tables@13.2.X)
3. Try to use that client to access the table: you get a "AuthorizationPermissionMismatch" error.

Expected behavior
Accessing the table should match the authorizations from the SAS key.

Additional context
This happens because of this line:

azure-sdk-for-js/sdk/tables/data-tables/src/utils/isCosmosEndpoint.ts

Line 14 in 83efc34

if (parsedURL.hostname === "localhost" && parsedURL.port !== "10002") {

Which basically assumes that if this is not the default emulator port it must be CosmoDB...

In our local setup we have cases where we want to emulate locally our data separation in multitenant mode by accessing different azure storage servers depending on tenant. So we need to be able to run multiple azurite instances on custom ports to achieve that.

[github-actions]

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @klaaslanghout.

[jeremymeng]

@cognot thanks for reporting the issue! I think we could probably check whether the url contains "devstoreaccount1" to determine if the endpoint is for tables

[jeremymeng]

Azurite also supports custom storage accounts, so we may need to check account name against AZURITE_ACCOUNTS env variables.