Skip to content

fix(auth): include OAuth resource in login flow#107

Merged
Mukil Loganathan (langchain-infra) merged 1 commit into
mainfrom
mukil/oauth-login-resource
May 6, 2026
Merged

fix(auth): include OAuth resource in login flow#107
Mukil Loganathan (langchain-infra) merged 1 commit into
mainfrom
mukil/oauth-login-resource

Conversation

@langchain-infra
Copy link
Copy Markdown
Contributor

Summary

  • include the normalized API origin as the OAuth resource during device-code, device-token, and refresh-token requests
  • cover resource handling in login and cmdutil refresh tests
  • document why the resource excludes the /api/v1 suffix

Verification

  • go test ./...
  • live dev login against https://dev.api.smith.langchain.com completed after device-code authorization
  • reran workspace/project checks with LangSmith env overrides unset so the saved OAuth profile was used

Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@langchain-infra Mukil Loganathan (langchain-infra) merged commit 1677c95 into main May 6, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants