-
Notifications
You must be signed in to change notification settings - Fork 3
Use OAuth in Claude #221
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use OAuth in Claude #221
Conversation
Add id-token: write and other necessary permissions to fix OIDC token authentication error in Claude GitHub Actions workflow.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Console Log Leak: Remove Internal URL
A debugging console.log
statement, logging injectedArgs.arbitratorJsonRpcUrl
, was accidentally left in src/components/disputeSummary.js
. This output is unrelated to the PR's OAuth changes and should be removed to prevent logging internal RPC URL information to the browser console.
src/components/disputeSummary.js#L40-L41
dispute-resolver/src/components/disputeSummary.js
Lines 40 to 41 in 4d8ef5d
console.log('injectedArgs.arbitratorJsonRpcUrl:', injectedArgs.arbitratorJsonRpcUrl); |
Was this report helpful? Give feedback by reacting with 👍 or 👎
@claude test |
Claude encountered an error —— View job
I'll analyze this and get back to you. |
No description provided.