-
Notifications
You must be signed in to change notification settings - Fork 323
fix: import OpenAPI 3.1/3.2 contentEncoding base64 as bytes with @encode decorator #9151
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
Conversation
…encode decorator Co-authored-by: baywet <[email protected]>
|
@copilot I'm getting the following error when building, fix it. |
…lity Co-authored-by: baywet <[email protected]>
... Fixed by casting the test schema to Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
commit: |
.chronus/changes/fix-base64-content-encoding-import-2025-12-5-18-32-55.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Timothee Guerin <[email protected]>
|
All changed packages have been documented.
Show changes
|
contentEncoding: base64should be imported asbytestype with@encode("base64", string)decoratorpackages/openapi3/src/cli/actions/convert/generate-types.ts(getStringTypefunction) to handlecontentEncoding: base64and returnbytestypedecorators.ts(getStringSchemaDecoratorsfunction) to add@encode("base64", string)decorator whencontentEncoding: base64Summary
This PR fixes the import of OpenAPI 3.1/3.2 schemas with
contentEncoding: base64.Before this fix:
Was incorrectly imported as:
After this fix:
Security Summary
The changes are minimal and focused on type detection during import. No security vulnerabilities were introduced.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.