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.

Sign up for GitHub

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

Jump to bottom

client, de-serialization does not work as response body of collection of BinaryData #2964

Open
weidongxu-microsoft opened this issue Oct 15, 2024 · 0 comments
Open

client, de-serialization does not work as response body of collection of BinaryData #2964

weidongxu-microsoft opened this issue Oct 15, 2024 · 0 comments
Labels
DPG

Comments

@weidongxu-microsoft
Copy link
Member

weidongxu-microsoft commented Oct 15, 2024 •
edited
Loading

ref microsoft/typespec#4669

Case of response body as e.g. unknown[] or Record<unknown>.
@weidongxu-microsoft weidongxu-microsoft mentioned this issue Oct 15, 2024
Merged
@weidongxu-microsoft weidongxu-microsoft changed the title client, de-serialization does not work on response of collection of BinaryData client, de-serialization does not work as response body of collection of BinaryData Oct 15, 2024
github-merge-queue bot pushed a commit to microsoft/typespec that referenced this issue Oct 18, 2024
@weidongxu-microsoft
http-client-java, unknown map to binary data (#4738)
27a0a4b 
Fix #4669

Added `use-object-for-unknown` to both swagger/typespec flag. Default is
`false`, meaning use `BinaryData`.
(we had 2 or 3 GA lib get affected, and may need to use this flag)

Pending issue Azure/autorest.java#2964
github-merge-queue bot pushed a commit to microsoft/typespec that referenced this issue Oct 18, 2024
@weidongxu-microsoft
http-client-java, unknown map to binary data (#4738)
31c42ef 
Fix #4669

Added `use-object-for-unknown` to both swagger/typespec flag. Default is
`false`, meaning use `BinaryData`.
(we had 2 or 3 GA lib get affected, and may need to use this flag)

Pending issue Azure/autorest.java#2964
github-merge-queue bot pushed a commit to microsoft/typespec that referenced this issue Oct 21, 2024
@weidongxu-microsoft
http-client-java, unknown map to binary data (#4738)
2d29598 
Fix #4669

Added `use-object-for-unknown` to both swagger/typespec flag. Default is
`false`, meaning use `BinaryData`.
(we had 2 or 3 GA lib get affected, and may need to use this flag)

Pending issue Azure/autorest.java#2964
swatkatz pushed a commit to swatkatz/typespec that referenced this issue Nov 5, 2024
@weidongxu-microsoft
http-client-java, unknown map to binary data (microsoft#4738)
a093daa 
Fix microsoft#4669

Added `use-object-for-unknown` to both swagger/typespec flag. Default is
`false`, meaning use `BinaryData`.
(we had 2 or 3 GA lib get affected, and may need to use this flag)

Pending issue Azure/autorest.java#2964
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
DPG
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@weidongxu-microsoft