Skip to content

Stop stripping explit query parameters from url in autorest emitter #3003

Merged
timotheeguerin merged 1 commit intoAzure:release/july-2025from
timotheeguerin:fix/query-paths
Jul 17, 2025
Merged

Stop stripping explit query parameters from url in autorest emitter #3003
timotheeguerin merged 1 commit intoAzure:release/july-2025from
timotheeguerin:fix/query-paths

Conversation

@timotheeguerin
Copy link
Copy Markdown
Member

@timotheeguerin timotheeguerin commented Jul 17, 2025

fix #2343

@microsoft-github-policy-service microsoft-github-policy-service Bot added the emitter:autorest Issues for @azure-tools/typespec-autorest emitter label Jul 17, 2025
@azure-sdk
Copy link
Copy Markdown
Collaborator

azure-sdk commented Jul 17, 2025

All changed packages have been documented.

  • @azure-tools/typespec-autorest
Show changes

@azure-tools/typespec-autorest - fix ✏️

Preserve explicit query parameters defined in the route by including them in x-ms-paths

@timotheeguerin timotheeguerin added the int:azure-specs Run integration tests against azure-rest-api-specs label Jul 17, 2025
@azure-sdk
Copy link
Copy Markdown
Collaborator

azure-sdk commented Jul 17, 2025

You can try these changes here

🛝 Playground 🌐 Website

@timotheeguerin timotheeguerin changed the title Experiement with fixes Stop stripping explit query parameters from url in autorest emitter Jul 17, 2025
@timotheeguerin timotheeguerin marked this pull request as ready for review July 17, 2025 16:37
@timotheeguerin
Copy link
Copy Markdown
Member Author

Validated manually against azure-rest-api-specs-pr as well and this cause no changes

@timotheeguerin timotheeguerin changed the base branch from main to release/july-2025 July 17, 2025 21:12
@timotheeguerin timotheeguerin enabled auto-merge (squash) July 17, 2025 21:21
@timotheeguerin timotheeguerin merged commit d3fa61d into Azure:release/july-2025 Jul 17, 2025
26 checks passed
@timotheeguerin timotheeguerin deleted the fix/query-paths branch July 17, 2025 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emitter:autorest Issues for @azure-tools/typespec-autorest emitter int:azure-specs Run integration tests against azure-rest-api-specs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Empty path generated by typespec-autorest emitter

3 participants