Skip to content
This repository has been archived by the owner on Nov 19, 2024. It is now read-only.

specify replica-signed queries #163

Merged
merged 50 commits into from
Sep 13, 2023
Merged

Conversation

mraszyk
Copy link
Contributor

@mraszyk mraszyk commented May 30, 2023

This PR adds public keys of nodes in a subnet to the certified state tree and specifies query call response signatures produced by nodes evaluating the query call and returned as part of the query response.

@mraszyk mraszyk requested a review from a team as a code owner May 30, 2023 21:14
@netlify
Copy link

netlify bot commented May 30, 2023

Deploy Preview for ic-interface-spec ready!

Name Link
🔨 Latest commit 11f5997
🔍 Latest deploy log https://app.netlify.com/sites/ic-interface-spec/deploys/64bfdc46dc298b0008a3f906
😎 Deploy Preview https://deploy-preview-163--ic-interface-spec.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

nomeata

This comment was marked as resolved.

@mraszyk

This comment was marked as resolved.

spec/requests.cddl Outdated Show resolved Hide resolved
spec/requests.cddl Outdated Show resolved Hide resolved
spec/requests.cddl Outdated Show resolved Hide resolved
spec/index.adoc Outdated Show resolved Hide resolved
spec/index.adoc Outdated Show resolved Hide resolved
@mraszyk mraszyk marked this pull request as draft June 7, 2023 05:30
@rumenov rumenov self-requested a review June 12, 2023 16:06
derlerd-dfinity

This comment was marked as outdated.

@ShuoWangNSL

This comment was marked as outdated.

@mraszyk

This comment was marked as outdated.

@nomeata

This comment was marked as resolved.

@ShuoWangNSL

This comment was marked as outdated.

@derlerd-dfinity

This comment was marked as outdated.

@ShuoWangNSL

This comment was marked as outdated.

@rumenov

This comment was marked as resolved.

@mraszyk

This comment was marked as outdated.

@mraszyk

This comment was marked as outdated.

@ShuoWangNSL ShuoWangNSL requested a review from DSharifi July 27, 2023 08:31
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
@netlify
Copy link

netlify bot commented Aug 9, 2023

Deploy Preview for ic-interface-spec ready!

Name Link
🔨 Latest commit 5e673b8
🔍 Latest deploy log https://app.netlify.com/sites/ic-interface-spec/deploys/64e4a50daa734d0008b7974e
😎 Deploy Preview https://deploy-preview-163--ic-interface-spec.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Dfinity-Bjoern Dfinity-Bjoern marked this pull request as ready for review August 16, 2023 09:29
Dfinity-Bjoern

This comment was marked as resolved.

spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
spec/index.md Outdated Show resolved Hide resolved
mraszyk and others added 4 commits September 1, 2023 12:19
Co-authored-by: Thomas Locher <thomas.locher@dfinity.org>
Co-authored-by: Thomas Locher <thomas.locher@dfinity.org>
gitlab-dfinity pushed a commit to dfinity/ic that referenced this pull request Sep 13, 2023
…de-signatures' into 'master'

feat: [IC-1247] Add replica signatures to Query responses

This MR adds replica signatures to Query responses.

A hash of a Query is computed as described in [PR#136](dfinity/interface-spec#163), signed with `sign_basic` and included in the query response.

Part of IC-1247
Closes NET-1500

Co-authored-by: Shuo Wang shuo.wang@dfinity.org \
Co-authored-by: Martin Raszyk martin.raszyk@dfinity.org 

Closes NET-1500

See merge request dfinity-lab/public/ic!13833
@mraszyk mraszyk merged commit b6fc9eb into master Sep 13, 2023
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants