feat: enable refresh token rotation for salesforce#1664
feat: enable refresh token rotation for salesforce#1664
Conversation
There was a problem hiding this comment.
Pull request overview
Enables Salesforce refresh token rotation by propagating an optionally returned refresh_token from the OAuth refresh exchange into the active ApiConnection session state, so subsequent refreshes use the latest token.
Changes:
- Extend refresh handling to accept an optional rotated refresh token and persist it in
sessionInfo. - Update the callout adapter refresh flow to capture
refresh_tokenfrom the refresh exchange and pass it through theonRefreshcallback. - Add structured debug/warn logs around token refresh attempts and failures.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| libs/salesforce-api/src/lib/connection.ts | Persist rotated refresh tokens into sessionInfo during refresh events. |
| libs/salesforce-api/src/lib/callout-adapter.ts | Plumb rotated refresh token through refresh exchange + callback; improve refresh logging. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ab4bd8c to
5f7e82a
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
5f7e82a to
47fb96e
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
47fb96e to
0b20eac
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
0b20eac to
0beda94
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
previously, we were not using the new refresh token if one was provided
0beda94 to
1a60190
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
previously, we were not using the new refresh token if one was provided