Skip to content
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

feat: Check that the sourceAccountIdentifier corresponds to account name (SCR-869) #1255

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

doubleface
Copy link
Contributor

@doubleface doubleface commented Oct 23, 2024

Clisk konnectors will now be allowed to have multiple accounts for the
same konnector.

To make it possible, the flagship app will allways check that the
sourceAccountIdentifier returned by the konnector, corresponds to the
accountName saved in the account. If there is a difference, it means
that the worker webview is authenticated with the wrong account and it
will ask to the konnector to authenticate again.

If the a wrong sourceAccountIdentifier is return twice, an error is
raised.

### ✨ Features

* Force logout when konnector identifier is different from account name (to allow multi-account for clisk konnectors)

Checklist

Before merging this PR, the following things must have been done if relevant:

  • Tested on iOS
  • Tested on Android
  • Test coverage
  • README and documentation

Clisk konnectors will now be allowed to have multiple accounts for the
same konnector.

To make it possible, the flagship app will allways check that the
sourceAccountIdentifier returned by the konnector, corresponds to the
accountName saved in the account. If there is a difference, it means
that the worker webview is authenticated with the wrong account and it
will ask to the konnector to authenticate again.

If the a wrong sourceAccountIdentifier is return twice, an error is
raised.
@doubleface doubleface force-pushed the feat/cliskMultiAccount branch from 48b25bc to 2055e10 Compare October 23, 2024 07:34
@doubleface doubleface merged commit f573d15 into master Oct 24, 2024
1 check failed
@doubleface doubleface deleted the feat/cliskMultiAccount branch October 24, 2024 09:35
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