Fixes issue #2208

[vigneashs]

This PR adds additional validation to authentication path computation for secondary requests.

[blueww]

@vigneashs

Thanks for raising the fix PR!
Would you please also add:

1. Change log: https://github.com/Azure/Azurite/blob/main/ChangeLog.md
2. (Optional) Test case: you can refer this case for how to add test case with product style Url:

   Azurite/tests/blob/specialnaming.test.ts

   Line 448 in 2fc3a7b

   it(`Should work with production style URL when ${productionStyleHostName} is resolvable`, async () => {

   
   (As I know, JS SDK only support secondary host name like "devstoreaccount1-secondary.blob.localhost", note the 2nd block need be "blob")

[blueww]

For the test case, I have drafted a test case in https://github.com/Azure/Azurite/pull/2216/files, for your reference.

[blueww]

@vinagesh

The following new case will fail, would you please help to look?

  1) table name validation tests
       Should work with production style URL when devstoreaccount1-secondary.table.localhost is resolvable:
     AssertionError [ERR_ASSERTION]: Failed
      at /home/vsts/work/1/s/tests/table/apis/table.validation.rest.test.ts:457:18
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)
      at async Context.<anonymous> (tests/table/apis/table.validation.rest.test.ts:441:5)

[vigneashs]

@vinagesh

The following new case will fail, would you please help to look?

  1) table name validation tests
       Should work with production style URL when devstoreaccount1-secondary.table.localhost is resolvable:
     AssertionError [ERR_ASSERTION]: Failed
      at /home/vsts/work/1/s/tests/table/apis/table.validation.rest.test.ts:457:18
      at runMicrotasks (<anonymous>)
      at processTicksAndRejections (internal/process/task_queues.js:95:5)
      at async Context.<anonymous> (tests/table/apis/table.validation.rest.test.ts:441:5)

I will fix it. Thanks

[vigneashs]

@blueww, I have fixed the error in Table unit tests.

The createStringToSignForSharedKeyLite routine had a hardcoded storage account name in authentication path which is valid only for non-production style URL. I have fixed that method to use both production style URL and non-production style URL.

[blueww]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[blueww]

@XiaoningLiu , @EmmaZhu
This PR looks good to me.
Would you please help to review this PR and see if you have any comments?
I plan to merge it in 1-2 days.