Attempting to delete table using storage account SAS with only Read and List permissions does not throw error

[craxal]

• Package Name: @azure/data-tables
• Package Version: 13.1.0
• Operating system: Windows 11
• nodejs
  • version: 16.13.1
• browser
  • name/version:
• typescript
  • version: 4.5.3
• Is the bug related to documentation in
  • README.md
  • source code documentation
  • SDK API docs on https://docs.microsoft.com

Describe the bug
When using a storage account SAS with Read and List permissions, attempting to delete a table no longer throws an error, despite the fact that a REST error is returned by the service. This is a regression from previous builds of the SDK.

To Reproduce

1. Run the following code snippet:

   // Make sure the SAS only has Read and List permissions, such as:
   // sv=2021-04-10&ss=btqf&srt=sco&st=2022-04-13T16%3A32%3A09Z&se=2022-04-14T16%3A32%3A09Z&sp=rl&sig=...
   const table = `testsasdel${process.hrtime.bigint()}`;
   const keyClient = TableClient.fromConnectionString(accounts.cralvordtest02.connectionString, table);
   const sasClient = TableClient.fromConnectionString("<SAS CONNECTION STRING>", table);
   
    try {
        console.log("Creating table...");
        await keyClient.createTable();
   
        console.log("Deleting table...");
        await sasClient.deleteTable();
   
        console.log("...done");
    } catch (error) {
        console.error(error);
    }

2. Observe the console output.

Expected behavior
A REST error should be output to the console and "...done" should not be output.

[joheredi]

It looks like we missed re-throwing the exception when the status code is not 404 when migrating to the new tracing version #20399. I'll send out a fix soon

[joheredi]

Fixed with PR #21408