Skip to content

Conversation

@dima74
Copy link
Contributor

@dima74 dima74 commented Jul 5, 2024

Description

When transfer domain from source account:

  • Fixed check in executor that authority owns domain (previously it checked that source owns domain, but it is not needed since it is checked in iroha_core)
  • Improved error message for check that source owns domain in iroha_core

Linked issue

Closes #4751

Benefits

Checklist

  • I've read CONTRIBUTING.md
  • I've used the standard signed-off commit format (or will squash just before merging)
  • All applicable CI checks pass (or I promised to make them pass later)
  • (optional) I've written unit tests for the code changes
  • I replied to all comments after code review, marking all implemented changes with thumbs up

@dima74 dima74 self-assigned this Jul 5, 2024
@dima74 dima74 force-pushed the diralik/fix-domain-transfer branch from 838240f to b713197 Compare July 5, 2024 15:08
nxsaken
nxsaken previously approved these changes Jul 5, 2024
Copy link
Contributor

@nxsaken nxsaken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only nits from me

@dima74 dima74 force-pushed the diralik/fix-domain-transfer branch from b713197 to 062711e Compare July 5, 2024 15:41
@dima74 dima74 force-pushed the diralik/fix-domain-transfer branch from 6a33d63 to 60fa87b Compare July 8, 2024 08:03
@dima74 dima74 merged commit 9174001 into hyperledger-iroha:main Jul 8, 2024
@dima74 dima74 deleted the diralik/fix-domain-transfer branch July 8, 2024 08:25
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.

[BUG] Domain transfer can be performed by anyone

3 participants