Skip to content

When using ChainableRamRoleArn profile and Oauth updated tokens are not saved to correct profile #75

Merged
yndu13 merged 1 commit intoaliyun:masterfrom
frco9:fix/oauth_token_save_wrong_profile
Jan 23, 2026
Merged

When using ChainableRamRoleArn profile and Oauth updated tokens are not saved to correct profile #75
yndu13 merged 1 commit intoaliyun:masterfrom
frco9:fix/oauth_token_save_wrong_profile

Conversation

@frco9
Copy link
Contributor

@frco9 frco9 commented Jan 19, 2026

Hello,

When using a ChainableRamRoleArn, and a source profile that is Oauth, the new tokens are saved to the first profile and not on the source_profile.

This PR is finding the source profile that has type OAuth and then save new config on it.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


FOUCAULT Jéremie seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@frco9
Copy link
Contributor Author

frco9 commented Jan 19, 2026

@yndu13 @CodeSpaceiiii can you have a look please ?

@yndu13
Copy link
Contributor

yndu13 commented Jan 20, 2026

Hello,

When using a ChainableRamRoleArn, and a source profile that is Oauth, the new tokens are saved to the first profile and not on the source_profile.

This PR is finding the source profile that has type OAuth and then save new config on it.

Thank you for the PR. We'll check and confirm it now.

@yndu13 yndu13 merged commit cf2bd22 into aliyun:master Jan 23, 2026
0 of 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.

3 participants