Azure / azure-sdk-for-net

This repository is for active development of the Azure SDK for .NET. For consumers of the SDK we recommend visiting our public developer docs at https://learn.microsoft.com/dotnet/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-net.
MIT License
5.36k stars 4.79k forks source link

[BUG]: AuthorizationScope is hardcoded in ContentSafetyClient #46395

Open arjaiswa opened 1 week ago

arjaiswa commented 1 week ago

Library name and version

Azure.AI.ContentSafety.ContentSafetyClient 1.0.0

Describe the bug

This is hardcoded in ContentSafetyClient private static readonly string[] AuthorizationScope = https://cognitiveservices.azure.com/.default;

Due to this we cannot use different base URL in Authorization scope https://cognitiveservices.azure.us/.default .

Expected behavior

Flkexbility to add required AuthorizationScope .

Actual behavior

No Flkexbility to add required AuthorizationScope

Reproduction Steps

Run ContentSafetyClient

Environment

No response

github-actions[bot] commented 1 week ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @bowgong @JieZhou000 @mengaims.

bowgong commented 6 days ago

@@.***>, please help make sure this fix is planned properly.

Thanks, Bowen

From: github-actions[bot] @.> Sent: Thursday, October 3, 2024 12:30 AM To: Azure/azure-sdk-for-net @.> Cc: Mention @.***> Subject: Re: [Azure/azure-sdk-for-net] [BUG]: AuthorizationScope is hardcoded in ContentSafetyClient (Issue #46395)

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @bowgonghttps://github.com/bowgong @JieZhou000https://github.com/JieZhou000 @mengaimshttps://github.com/mengaims.

- Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-sdk-for-net/issues/46395#issuecomment-2389104848 or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANOGEPGSTU3NXQWIKHPN7HTZZQNR5BFKMF2HI4TJMJ2XIZLTTGBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVIYTANRZGQZTINZQG6SG4YLNMWUWQYLTL5WGCYTFNSBKK5TBNR2WLKRRGE3TQNBUG44DMMFENZQW2ZNJNBQXGX3MMFRGK3ECUV3GC3DVMWVDCMZVGMZTCNRRHAZ2I3TBNVS2S2DBONPWYYLCMVWIFJLWMFWHKZNKGEZTMOJWHA3TSNBVURXGC3LFVFUGC427NRQWEZLMQKSXMYLMOVS2UMJZGM3DSNJWGYZTRJDOMFWWLKLIMFZV63DBMJSWZAVFOZQWY5LFVI3DMMJUHA3TMNZRHCSG4YLNMWUWQYLTL5WGCYTFNSWHG5LCNJSWG5C7OR4XAZNMJFZXG5LFINXW23LFNZ2KM5DPOBUWG44YQKSHI6LQMWVHEZLQN5ZWS5DPOJ42K5TBNR2WLJZSHEZDQOJUGSBKI5DZOBS2K2LTON2WLJLWMFWHKZNKGI2TMMJSHE3DOMZTQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKRRGA3DSNBTGQ3TAN4CUR2HS4DFUVWGCYTFNSSXMYLMOVS2UMJRG44DINBXHA3DBAVEOR4XAZNFNRQWEZLMUV3GC3DVMWVDCMZVGMZTCNRRHAZYFJDUPFYGLJLMMFRGK3FFOZQWY5LFVIYTGNRZGY4DOOJUGWBKI5DZOBS2K3DBMJSWZJLWMFWHKZNKGE4TGNRZGU3DMMZYQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKRWGYYTIOBXGY3TCOFHORZGSZ3HMVZKMY3SMVQXIZI. You are receiving this email because you were mentioned.

Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.