Good morning!

While debugging refresh token issues, I found that the vendor library doesn't store the refresh token in the access token it returns. This would result in the refresh only working once.

I have a nice, simple patch, though it will be a couple of days before I can say with authority that it makes everything work as it ought.

Patch coming after I've posted this issue.

CommentFileSizeAuthor
#3 Issue-3163385-3-provider-refreshAccessToken-doesn-t-ke.patch1.35 KBnigelcunningham
#2 Issue-3163385-provider-refreshAccessToken-doesn-t-ke.patch992 bytesnigelcunningham

Comments

NigelCunningham created an issue. See original summary.

nigelcunningham’s picture

nigelcunningham commented
StatusFileSize
new Issue-3163385-provider-refreshAccessToken-doesn-t-ke.patch992 bytes
nigelcunningham’s picture

nigelcunningham commented
StatusFileSize
new Issue-3163385-3-provider-refreshAccessToken-doesn-t-ke.patch1.35 KB

Updated patch, moving the retention of the refresh token into the storage class.

nigelcunningham’s picture

nigelcunningham commented
Status: Active » Needs review
aaronbauman’s picture

aaronbauman
he/him
commented
Status: Needs review » Reviewed & tested by the community

Wow, thank you for getting to the bottom of this.
I have a feeling this will address several other OAuth-related issues.

Just the latest in a long line of annoyances with Salesforce API and OAuth in general.
Will have this committed shortly and rolled into a new release soon.

aaronbauman’s picture

aaronbauman
he/him
commented
Status: Reviewed & tested by the community » Fixed

Committed, thanks again.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.