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

Add MetadataTransaction targetPublicKey opt-in notification #50

Closed
dgarcia360 opened this issue Oct 25, 2019 · 1 comment
Closed

Add MetadataTransaction targetPublicKey opt-in notification #50

dgarcia360 opened this issue Oct 25, 2019 · 1 comment

Comments

@dgarcia360
Copy link
Member

@dgarcia360 dgarcia360 commented Oct 25, 2019

1. Expected behavior and actual behavior.

The account, namespace, or mosaic owner must opt-in to all metadata requests received by giving explicit permission. In practice, this means that all MetadataTransactions must be wrapped in an AggregateTransaction.

However, the account set as targetPublicKey is not receiving a notification through the partial channel (WebSocket) nor the partial transactions endpoint.

2. Steps to reproduce the problem.

Step 1: B wants to add a metadata entry to account A (code)

Step 2: Try to retrieve the transaction calling /account/{accountId}/transactions/partial endpoint, using A's public key.

3. Specifications like the version of the project, operating system, or hardware.

  • catapult-server 0.8.0.3
gimer pushed a commit that referenced this issue Nov 8, 2019
### Added
 - Support for `make install`

### Changed
 - Add hash variant name to binary descriptions
 - Realign transaction binary layouts to maximize alignment of fields
 - Add `AggregateTransaction::TransactionHash` that contains merkle hash of component transactions
 - Store `Version` and `Network` directly in `EntityBody`

### Fixed
 - GitHub Issues: #45 #46 #47 #48 #49 #50
 - Bug in harvester `StateHash` calculation when `transactionSelectionStrategy` is not `oldest`
 - Fix remaining UBSAN and TSAN warnings
@Jaguar0625

This comment has been minimized.

Copy link
Member

@Jaguar0625 Jaguar0625 commented Nov 8, 2019

Fixed in 621ccbd [v0.9.0.1].

@Jaguar0625 Jaguar0625 closed this Nov 8, 2019
Jaguar0625 pushed a commit that referenced this issue Nov 14, 2019
### Added
 - Support for `make install`

### Changed
 - Add hash variant name to binary descriptions
 - Realign transaction binary layouts to maximize alignment of fields
 - Add `AggregateTransaction::TransactionHash` that contains merkle hash of component transactions
 - Store `Version` and `Network` directly in `EntityBody`

### Fixed
 - GitHub Issues: #45 #46 #47 #48 #49 #50
 - Bug in harvester `StateHash` calculation when `transactionSelectionStrategy` is not `oldest`
 - Fix remaining UBSAN and TSAN warnings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.