feat(sdk): add gateway, auth, fakes, and edge client (2/4)#10
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
8ec90d2 to
b5db057
Compare
Add the gateway client (reads CLI config), OIDC authentication with token refresh, edge/tunnel client, and in-memory fake implementations for testing. Part 2 of 4 for issue NVIDIA#2044. Assisted-By: 🤖 Claude Code Signed-off-by: Roland Huß <rhuss@redhat.com>
b5db057 to
edbf3ed
Compare
|
@coderabbitai review |
✅ Action performedReview finished.
|
Applied fixes from bot review comments: - Comment #3563989616: add nil check to fakeProviderClient.Create - Comment #3563989625: add nil check to fakeProviderClient.Update - Comment #3563989635: add nil check to fakeProviderClient.Ensure - Comment #3563989656: fix misleading comment about logging readErr in oidc.Login Assisted-By: 🤖 Claude Code Signed-off-by: Roland Huß <rhuss@redhat.com>
Lower the minimum Go version to 1.24.0, matching PR #9. Assisted-By: 🤖 Claude Code
Lower the minimum Go version to 1.24.0, matching PR #9. Assisted-By: 🤖 Claude Code
2749499 to
552e801
Compare
Lower the minimum Go version to 1.24.0, matching PR #9. Assisted-By: 🤖 Claude Code
552e801 to
db75560
Compare
Summary
Add the gateway client (reads CLI config), OIDC authentication with token refresh, edge/tunnel client, and in-memory fake implementations for testing.
Part 2 of 4 for #2044. Depends on PR 1 (foundation).
What's Included
gateway/): Reads~/.config/openshell/config, resolves auth automaticallyoidc/): Discovery, device-code flow, auth-code flow, token managementedge/): Cloudflare tunnel integration for edge API accessfake/): In-memory fakes mirroring real client validation for testingFile Tree
70 files