azure-rest-api-specs icon indicating copy to clipboard operation
azure-rest-api-specs copied to clipboard

Azure Container Apps session data plane

Open yitaopan opened this issue 1 year ago • 3 comments

Choose a PR Template

Switch to "Preview" on this description then select one of the choices below.

Click here to open a PR for a Data Plane API.

Click here to open a PR for a Control Plane (ARM) API.

fix https://github.com/Azure/azure-rest-api-specs/issues/30393

yitaopan avatar Sep 04 '24 09:09 yitaopan

Next Steps to Merge

✅ All automated merging requirements have been met! To get your PR merged, see aka.ms/azsdk/specreview/merge.

Primary feedback is to reconsider the URL structure in API spec which would impact the TypeSpec and SDK surfaces.

A few other considerations for you to think about:

  1. Potential LRO & idempotency on the CodeExecute API.
  2. File upload functionality. Is it possible to leverage Azure Storage? If you are internally provisioning storage acct, can you expose a timed storage SAS url for user to interact with using Azure Storage API/SDK, rather than handling list/upload/download & file management yourself?

allenjzhang avatar Sep 05 '24 17:09 allenjzhang