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

Clarification on how to serialize NaN, Infinity, and NegInfinity for AWS protocols #812

Closed
jdisanti opened this issue May 27, 2021 · 2 comments · Fixed by #828
Closed

Clarification on how to serialize NaN, Infinity, and NegInfinity for AWS protocols #812

jdisanti opened this issue May 27, 2021 · 2 comments · Fixed by #828
Assignees
Labels
documentation This is a problem with documentation. protocol-test New protocol tests are needed

Comments

@jdisanti
Copy link
Contributor

I wasn't able to find information in the specifications on how to handle NaN or Infinity values for floating points for either serialization or deserialization of the protocols. What is the expected behavior for these values?

@JordonPhillips
Copy link
Contributor

That's a good question. I think this will have to be protocol specific behavior. I'm still looking into this, but it seems that the JSON protocols don't support those values (as JSON doesn't). restXML appears to accept them and respond to them fine, but it's unclear whether any service does anything with them (the answer is almost certainly that they don't). Currently unsure about awsQuery / ec2Query

@JordonPhillips JordonPhillips added documentation This is a problem with documentation. protocol-test New protocol tests are needed labels May 31, 2021
@JordonPhillips JordonPhillips self-assigned this Jun 8, 2021
@JordonPhillips
Copy link
Contributor

I've looked into this, and it looks like existing services theoretically can support these special values when handled as a string. I'll work on a PR to add these explicitly to the spec.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This is a problem with documentation. protocol-test New protocol tests are needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants