Skip to content

Do not join ? and : with /#9935

Open
timotheeguerin wants to merge 9 commits intomicrosoft:mainfrom
timotheeguerin:fix/http-no-join-query-and-colon
Open

Do not join ? and : with /#9935
timotheeguerin wants to merge 9 commits intomicrosoft:mainfrom
timotheeguerin:fix/http-no-join-query-and-colon

Conversation

@timotheeguerin
Copy link
Member

@timotheeguerin timotheeguerin commented Mar 5, 2026

fix #9936

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 5, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@typespec/http@9935
npm i https://pkg.pr.new/@typespec/rest@9935

commit: bb98009

@timotheeguerin timotheeguerin added the int:azure-specs Run integration tests against azure-rest-api-specs label Mar 5, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

All changed packages have been documented.

  • @typespec/http
Show changes

@typespec/http - fix ✏️

Do not join routes starting with ? or : with /(e.g. @route("?pet=cat) would result in /?pet=cat)

@azure-sdk
Copy link
Collaborator

azure-sdk commented Mar 5, 2026

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

`
@route("abc")
interface Container {
@route("?restype=container") foo(): void;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably need to test with two query parameters joined with & as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

int:azure-specs Run integration tests against azure-rest-api-specs lib:http lib:rest

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@route starting with ? and : shouldn't be joined with /

4 participants