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

New icq packet format #11

Open
wants to merge 1 commit into
base: feature/icq_integration
Choose a base branch
from

Conversation

0xLaurenzo
Copy link

@0xLaurenzo 0xLaurenzo commented Sep 29, 2022

Changes the icq packet format. Also this removes auth of query types from ICQ for easy testing
The packet format changes from the old version:

// InterchainQueryPacketData is comprised of an ABCI query request and type of query.
message InterchainQueryPacketData {
  repeated tendermint.abci.RequestQuery requests = 1 [(gogoproto.nullable) = false];
}

// InterchainQueryPacketAck is comprised of an ABCI query response with non-deterministic fields left empty (e.g. Codespace, Log, Info and ...).
message InterchainQueryPacketAck {
  repeated tendermint.abci.ResponseQuery responses = 1 [(gogoproto.nullable) = false];
}

to the new version:

// InterchainQueryPacketData is comprised of raw query.
message InterchainQueryPacketData {
  bytes data = 1;
}

// InterchainQueryPacketAck is comprised of an ABCI query response with non-deterministic fields left empty (e.g. Codespace, Log, Info and ...).
message InterchainQueryPacketAck {
  bytes data = 1;
}

// CosmosQuery contains a list of tendermint ABCI query requests. It should be used when sending queries to an SDK host chain.
message CosmosQuery {
  repeated tendermint.abci.RequestQuery requests = 1 [(gogoproto.nullable) = false];
}

// CosmosResponse contains a list of tendermint ABCI query responses. It should be used when receiving responses from an SDK host chain.
message CosmosResponse {
  repeated tendermint.abci.ResponseQuery responses = 1 [(gogoproto.nullable) = false];
}

@Ehsan-saradar feel free to merge once qoracle is ready for this aswell

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.

1 participant