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

Feature/cloudproof rust #94

Merged
merged 2 commits into from
Mar 1, 2023
Merged

Feature/cloudproof rust #94

merged 2 commits into from
Mar 1, 2023

Conversation

tbrezot
Copy link
Collaborator

@tbrezot tbrezot commented Feb 8, 2023

Open question: serde and serde_json are needed only for serializing/deserializing Policy; is it justified?

We could write our own serialization functions and drop serde dependency... but breaking backward compatibility (which has been broken anyway by the last release in almost all client libs)

Closes #98

@tbrezot tbrezot marked this pull request as draft February 8, 2023 15:28
ThibaudDauce
ThibaudDauce previously approved these changes Feb 8, 2023
Cargo.lock Outdated Show resolved Hide resolved
rustfmt.toml Outdated Show resolved Hide resolved
src/error.rs Show resolved Hide resolved
@tbrezot tbrezot marked this pull request as ready for review February 20, 2023 12:40
feat!: use 16-bytes tag

fix: bump version and fills changelog

fix!: do not use serde where it can be avoided

fix: use CryptoCore from GitHub branch

fix: corrections

fix: formatting

feat: remove `.cargo/`

fix: remove `println!`

feat: improve handling of unknown axis error

fix: update CryptoCore

fix: examples

feat: use `Shake256` from `tiny_keccak`

fix: non regression tests

fix: CHANGELOG

fix!: use crypto_core 7.0

fix: use CryptoCore public constants and types

fix: format
@Manuthor Manuthor force-pushed the feature/cloudproof_rust branch 4 times, most recently from dd26dc9 to 14192c3 Compare March 1, 2023 08:43
@Manuthor Manuthor merged commit d0a531c into develop Mar 1, 2023
@Manuthor Manuthor deleted the feature/cloudproof_rust branch March 1, 2023 14:56
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.

3 participants