Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UserGetSingleton Type Error #419

Closed
tgoodsell-tempus opened this issue Nov 29, 2023 · 4 comments
Closed

UserGetSingleton Type Error #419

tgoodsell-tempus opened this issue Nov 29, 2023 · 4 comments
Labels
bug Something isn't working stale

Comments

@tgoodsell-tempus
Copy link

Describe the bug?

UserGetSingleton has an incorrect type for its type field it appears. Based on the documentation, this is returning a error upstream such as: okta/terraform-provider-okta#1836

What is expected to happen?

Property unmarshalls to the GoStruct successfully

What is the actual behavior?

Does not happen

Reproduction Steps?

See the Terraform Provider Issue

Additional Information?

https://github.com/okta/okta-sdk-golang/blob/846efbd8f9f45f3bf295acf2ff4e3a2ea63a9e3b/okta/docs/UserGetSingleton.md?plain=1#L18C2-L18C2

Golang Version

go version go1.21.4 darwin/arm64

SDK Version

3.0.18

OS version

No response

@tgoodsell-tempus tgoodsell-tempus added the bug Something isn't working label Nov 29, 2023
@tgoodsell-tempus
Copy link
Author

This seems to be because the spec used to generate this is incorrect compared to upstream:

Bad spec: https://github.com/okta/okta-sdk-golang/blob/65eaabab4de5851f3d66a2068003842290b9df85/.generator/okta-management-APIs-oasv3-enum-inheritance.yaml#L35902C7-L35902C7

Upstream spec: https://github.com/okta/okta-management-openapi-spec/blob/61659e78d1ff598ac43766f4c7e969080e6c6e15/resources/3.0/management.yaml#L27782

I don't want to try and DIY a fix since I have no idea if we should be using the v3 spec at all (the upstream repo documentation indicates the v2 as the primary spec) nor do I understand why this spec is pulled into this repo at all.

Copy link

This issue has been marked stale because there has been no activity within the last 14 days. To keep this issue active, remove the stale label.

@github-actions github-actions bot added the stale label Dec 14, 2023
@monde monde removed the stale label Jan 16, 2024
Copy link

This issue has been marked stale because there has been no activity within the last 14 days. To keep this issue active, remove the stale label.

@github-actions github-actions bot added the stale label Jan 31, 2024
@duytiennguyen-okta
Copy link
Contributor

This is fixed in v3.0.19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working stale
Projects
None yet
Development

No branches or pull requests

3 participants