Skip to content

[python] noAuth results in optional cred parameter - #9298

Merged
iscai-msft merged 2 commits into
microsoft:mainfrom
iscai-msft:python/optionalCred
Jan 13, 2026
Merged

[python] noAuth results in optional cred parameter#9298
iscai-msft merged 2 commits into
microsoft:mainfrom
iscai-msft:python/optionalCred

Conversation

@iscai-msft

@iscai-msft iscai-msft commented Jan 6, 2026 •
edited by msyyc
Loading

Copy link
Copy Markdown
Member

@microsoft-github-policy-service microsoft-github-policy-service Bot added the emitter:client:python Issue for the Python client emitter: @typespec/http-client-python label Jan 6, 2026
@pkg-pr-new

pkg-pr-new Bot commented Jan 6, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/microsoft/typespec/@typespec/http@9298

npm i https://pkg.pr.new/microsoft/typespec/@typespec/http-client@9298

commit: 593f045

@github-actions

github-actions Bot commented Jan 6, 2026

Copy link
Copy Markdown
Contributor

All changed packages have been documented.

  • ✅ @typespec/http-client-python
Show changes

@typespec/http-client-python - fix ✏️

Fix clients with NoAuth credentials to have optional generated credential parameters

@azure-sdk

Azure SDK Bot (azure-sdk) commented Jan 6, 2026

Copy link
Copy Markdown
Collaborator

You can try these changes here

🛝 Playground 🌐 Website 🛝 VSCode Extension

pvaneck
Paul Van Eck (pvaneck) approved these changes Jan 6, 2026
View reviewed changes

@pvaneck Paul Van Eck (pvaneck) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks! Tested with my use case, and the client no longer requires a credential anymore.

@iscai-msft
iscai-msft added this pull request to the merge queue Jan 13, 2026
Merged via the queue into microsoft:main with commit 89ef757 Jan 13, 2026
38 checks passed
@iscai-msft
iscai-msft deleted the python/optionalCred branch January 13, 2026 19:58
@grdsdev Guilherme Souza (grdsdev) mentioned this pull request Oct 22, 2026
Open
4 tasks done
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tadelesh Chenjie Shi (tadelesh) tadelesh approved these changes
@msyyc Yuchao Yan (msyyc) msyyc approved these changes
@timotheeguerin Timothee Guerin (timotheeguerin) Awaiting requested review from timotheeguerin timotheeguerin is a code owner
@lmazuel Laurent Mazuel (lmazuel) Awaiting requested review from lmazuel lmazuel is a code owner
@swathipil Swathi Pillalamarri (swathipil) Awaiting requested review from swathipil
@catalinaperalta catalinaperalta Awaiting requested review from catalinaperalta catalinaperalta is a code owner
@laiapat Laia Patiño (laiapat) Awaiting requested review from laiapat
+1 more reviewer
@pvaneck Paul Van Eck (pvaneck) pvaneck approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

emitter:client:python Issue for the Python client emitter: @typespec/http-client-python

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug]: NoAuth with OAuth2Auth emits Client that still requires credentials

5 participants

@iscai-msft @azure-sdk @tadelesh @pvaneck @msyyc