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 StructuredEncryption smithy model and stubbed APIs #1

Merged
merged 13 commits into from
Aug 24, 2022

Conversation

lavaleri
Copy link
Contributor

@lavaleri lavaleri commented Jul 9, 2022

Issue #, if available:

Description of changes:

Define the Smithy model and creates stubs for the new StructuredEncryption APIs. Added minimal input validation; when we come in with the spec/duvet next we can ensure that this input validation is complete. Code is verified and the HappyTestCase can successfully run via .NET.

I removed the generated C# files, as Polymorph doesn't handle unions in C# correctly. Thus there is no E2E C# examples.

The build structure and tooling is still WIP, and likely to change as we make progress on the MPL. For now, it is adequate for verifying/building/running this code in Dafny.

Testing done via:

make verify && make build && make test

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@lavaleri lavaleri marked this pull request as draft July 9, 2022 00:52
@lavaleri lavaleri marked this pull request as ready for review August 22, 2022 23:59
Makefile Show resolved Hide resolved
.gitmodules Show resolved Hide resolved
Copy link
Contributor

@texastony texastony left a comment

Choose a reason for hiding this comment

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

Looks groovy.
I do not know what is going on with History
or the Modifies stuff.
I have a few nitpicks and questions.

@lavaleri lavaleri merged commit 7f8f8e7 into main Aug 24, 2022
@lavaleri lavaleri deleted the structured-encryption-model branch August 24, 2022 17:50
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.

2 participants