Skip to content

Fix accept transfer - #2475

Merged
sk-keeper merged 2 commits into
Keeper-Security:releasefrom
lthievenaz-keeper:fix-acceptTransfer
Oct 29, 2026
Merged

sk-keeper merged 2 commits into
Keeper-Security:releasefrom
lthievenaz-keeper:fix-acceptTransfer

Conversation

@lthievenaz-keeper

Copy link
Copy Markdown
Contributor

The accept-transfer command cannot show a successful result. This is because it expects a {"result_code":"success"} response, when the key is actually "result":

{'result': 'success', 'result_code': '', 'message': '', 'command': 'share_account'}

accept-transfer always fail due to incorrect condition looking at the API result.
"Account transfer accepted" and "Account transfer canceled" is misleading and may suggest an actual transfer-user action, when it just relates to transfer policy acceptance.

Aligned the wording to be the same as api.py
@sk-keeper
sk-keeper merged commit cb21e9a into Keeper-Security:release Oct 29, 2026
4 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.

2 participants