azure-sdk-for-go icon indicating copy to clipboard operation
azure-sdk-for-go copied to clipboard

Read Only Fields in DenyAssignmentProperties and Missing Clients

Open wenjungaogaogao opened this issue 2 years ago • 5 comments

Bug Report

  • import path of package in question:

    • "github.com/Azure/azure-sdk-for-go/services/preview/authorization/mgmt/2018-09-01-preview/authorization"
    • And other apiversions as well.
  • SDK version e.g. master

    • master
  • output of go version

    • go version go1.18.2 linux/amd64
  • What happened?

    1. The principal field is marked as read only
    2. There's no sdk for denyAssignments create and list. List is public preview while create can only be performed by Azure internal. Details here.
  • What did you expect or want to happen?

    1. Making the principal field non read-only. Now, upon put we need to copy the definition of DenyAssignmentProperties as otherwise the principal field will be dropped when MarshalJSON().
    2. Now we have to put/get deny assignments via generic armresource client. It'd be great if it can have a dedicated client in track 2.

wenjungaogaogao avatar Aug 08 '22 22:08 wenjungaogaogao

@wenjungaogaogao We have deprecated github.com/Azure/azure-sdk-for-go/services/preview/authorization/mgmt/2018-09-01-preview/authorization. A replacement package is available github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/authorization/armauthorization. See Migration Guide for guidance on upgrading. Refer to our deprecation policy for more details.

tadelesh avatar Aug 09 '22 09:08 tadelesh

@tadelesh Thanks for the quick response. However DenyAssignment related model and client are not available in the auth sdk.

wenjungaogaogao avatar Aug 10 '22 18:08 wenjungaogaogao

@Alancere Please help to release a new version for authorization with tag package-2022-04-01.

tadelesh avatar Aug 16 '22 08:08 tadelesh

Meanwhile, @wenjungaogaogao feel free to share your experience using the Azure SDK in this survey.

tadelesh avatar Aug 24 '22 07:08 tadelesh

Please try new version: https://pkg.go.dev/github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/authorization/armauthorization/[email protected]

tadelesh avatar Sep 27 '22 06:09 tadelesh