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

Fix DescribeAvailablePatches api #4043

Closed
wants to merge 1 commit into from

Conversation

svagner
Copy link

@svagner svagner commented Aug 3, 2021

Epoch is returned by api is string type not int64

aws/aws-sdk-go-v2#2655

Epoch is returned by api is string type not int64
@jasdel
Copy link
Contributor

jasdel commented Sep 20, 2021

Thanks for reporting this issue @svagner, and providing a solution for it. The SDK cannot take this change directly since these model files are copied into the SDK automatically. The copy would overwrite this change. In addition, this change breaks the SDK's public released type. Changing it from a int to string.

We'll need to forward this issue to the API team, to correct their API's behavior. This most likely is an issue with the service sending the wrong type in this specific case.

@jasdel jasdel added service-api This issue is due to a problem in a service API, not the SDK implementation. bug This issue is a bug. labels Sep 21, 2021
@jasdel
Copy link
Contributor

jasdel commented Sep 21, 2021

Lets use the linked GitHub Issue, aws/aws-sdk-go-v2#2655 created to track this issue and resolution.

@jasdel jasdel closed this Sep 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. service-api This issue is due to a problem in a service API, not the SDK implementation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants