Removing the signed-resource requirement for account SAS tokens

[tomswedlund]

Addressing the following bug: #383

[Jinming-Hu]

/azp run

[azure-pipelines]

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

[Jinming-Hu]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[azure-pipelines]

Commenter does not have sufficient privileges for PR 396 in repo Azure/azure-storage-cpp

[tomswedlund]

Hi Jinming,

Sorry for the build break, I swear I built this locally. But this brings up a different issue. Should the conditional that I removed actually be removed, or should we be passing false to the function in case an account SAS is being used? Looks like "require_signed_resource" is only set to true in two places:

• src/cloud_file_directory.cpp, line 72
• src/cloud_file.cpp, line 92

Should one of these (or both) be set to false instead?

Tom

[Jinming-Hu]

@tomswedlund Hi, thanks for your contribution. I believe the CI failure has nothing to do with the change in this PR. I'm gonna take some time to fix Azure pipelines.

[tomswedlund]

HI Jinming,

I see a build failure with the following error:
microsoft.windowsazure.storage\src\shared_access_signature.cpp(85,0): Error C4100: 'require_signed_resource': unreferenced formal parameter

Looks like the 'require_signed_resource' function param is no longer being used because of my change, and hence the error. Should this parameter be removed altogether? Or should we passing 'false' in the two places I mentioned in my last comment?

Thanks,
Tom