search

Azure / azure-sdk-for-js

This repository is for active development of the Azure SDK for JavaScript (NodeJS & Browser). For consumers of the SDK we recommend visiting our public developer docs at https://docs.microsoft.com/javascript/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-js.
MIT License
2.03k stars 1.19k forks source link

[Epic] Refresh all the rlc sdks in sdk repo #30680

Open kazrael2119 opened 1 month ago

kazrael2119 commented 1 month ago

part of the followup on https://github.com/Azure/azure-sdk-for-js/issues/29883#issuecomment-2150188495

### Data-Plane
- [ ] [sdk\agrifood\agrifood-farming-rest](https://github.com/Azure/azure-sdk-for-js/pull/30779)
- [ ] sdk\anomalydetector\ai-anomaly-detector-rest (no need)
- [ ] [sdk\communication\communication-job-router-rest](https://github.com/Azure/azure-sdk-for-js/pull/30907)
- [ ] [sdk\communication\communication-messages-rest](https://github.com/Azure/azure-sdk-for-js/pull/30896)
- [ ] [sdk\confidentialledger\confidential-ledger-rest](https://github.com/Azure/azure-sdk-for-js/pull/30909)
- [ ] [sdk\contentsafety\ai-content-safety-rest](https://github.com/Azure/azure-sdk-for-js/pull/30925)
- [ ] [sdk\devcenter\developer-devcenter-rest](https://github.com/Azure/azure-sdk-for-js/pull/30936)
- [ ] [sdk\deviceupdate\iot-device-update-rest](https://github.com/Azure/azure-sdk-for-js/pull/30963)
- [ ] sdk\documentintelligence\ai-document-intelligence-rest(no need as service team have released a new version)
- [ ] [sdk\documenttranslator\ai-document-translator-rest](https://github.com/Azure/azure-sdk-for-js/pull/30967)
- [ ] [sdk\easm\defender-easm-rest](https://github.com/Azure/azure-sdk-for-js/pull/30976)
- [ ] [sdk\face\ai-vision-face-rest](https://github.com/Azure/azure-sdk-for-js/pull/30978)
- [ ] sdk\healthinsights\health-insights-cancerprofiling-rest
- [ ] sdk\healthinsights\health-insights-clinicalmatching-rest
- [ ] sdk\healthinsights\health-insights-radiologyinsights-rest
- [ ] [sdk\loadtesting\load-testing-rest](https://github.com/Azure/azure-sdk-for-js/pull/31029)
- [ ] sdk\maps\maps-geolocation-rest
- [ ] sdk\maps\maps-render-rest
- [ ] sdk\maps\maps-route-rest
- [ ] sdk\maps\maps-search-rest
- [ ] sdk\purview\purview-administration-rest
- [ ] sdk\purview\purview-catalog-rest
- [ ] sdk\purview\purview-datamap-rest
- [ ] sdk\purview\purview-scanning-rest
- [ ] sdk\purview\purview-sharing-rest
- [ ] sdk\purview\purview-workflow-rest
- [ ] sdk\synapse\synapse-access-control-rest
- [ ] sdk\translation\ai-translation-document-rest
- [ ] sdk\translation\ai-translation-text-rest
### Mgmt-Plane
- [ ] sdk\appservice\arm-appservice-rest
- [ ] sdk\compute\arm-compute-rest
- [ ] sdk\containerservice\arm-containerservice-rest
- [ ] sdk\network\arm-network-rest
- [ ] sdk\servicefabric\arm-servicefabric-rest
qiaozha commented 1 month ago

@xirzec I think we can come back to refresh the RLC libraries for all both data plane and mgmt plane so that we can proceed on this https://github.com/Azure/azure-sdk-for-js/pull/29904 after it finishes. Wonder if you have any concerns about our side refresh the data plane libraries?

qiaozha commented 1 month ago

@MaryGao @timovv Could you provide more context on why we need to refresh the RLC for multipart form data support?

timovv commented 1 month ago

My main concern is that the non-refreshed SDKs are still using @azure-rest/core-client version 1.x.x, which is no longer receiving updates. There should be no breaking change for RLCs which don't use multipart/form-data, but for those that do use multipart/form-data (I don't know how many there are, maybe one or two max?) the refresh will result in a breaking change, as we agreed upon when we finalized the design.