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

chore(.NET): Add ESDK-Net v4.0.1 generated vectors #636

Merged
merged 3 commits into from
Feb 13, 2024

Conversation

imabhichow
Copy link
Contributor

Issue #, if available:

Description of changes:

  • Add vectors generated by ESDK.NET v4.0.1 using the v4 manifest and update the CI to run vectors

Squash/merge commit message, if applicable:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.


- name: Run ESDK-NET @ v4.0.1 Vectors expect success
working-directory: ./AwsEncryptionSDK/runtimes/net/TestVectorsNative/TestVectors
continue-on-error: true
Copy link
Contributor

Choose a reason for hiding this comment

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

sorry I missed this, why are we continuing on error? Shouldn't the run fail id we fail to decrypt all vectors when we expect everything to pass?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I think I forgot to remove this. I'll update the PR.

@imabhichow imabhichow merged commit efef497 into mainline Feb 13, 2024
7 checks passed
@imabhichow imabhichow deleted the imabhichow/add-net4.0.1-vectors branch February 13, 2024 00:42
rishav-karanjit added a commit that referenced this pull request May 9, 2024
## 4.1.0

### Notes
- [(#646)](10daadf) Enforces input constraints.

Prior to this fix, the AWS Encryption SDK in .NET (ESDK-NET) failed to enforce user input constraints. Input shapes without required members set would always result in a `NullReferenceException`.
Now, the ESDK-NET will throw its own Exceptions when illegal user input
is submitted.

### Fixes

* fix: throw an exception when MemoryStream instance has an empty backing array [(#633)](550c714)

### Features

* feat: enforce input constraints [(#646)](10daadf)

### Maintenance

* fix(CI): Daily CI uses correct workflow [(#641)](771835e)
* chore(ci): fix role to assume [(#622)](c1f04fc)
* chore(CI/CD): add semantic release automation [(#647)](e7b5392)
* chore: Adopt SmithyDafnyMakefile.mk, fix nightly build [(#638)](cd19979)
* chore(CI): add interop tests to daily ci [(#640)](c9ad018)
* chore: only run net48 on windows and use node 17 to run integration-node [(#639)](d6c62fb)
* chore(.NET): Add ESDK-Net v4.0.1 generated vectors[(#636)](efef497)
* chore(NET-SupportPolicy): Mark 3.x as Support [(#631)](3c36f7a)
* chore: Add manual trigger for nightly_dafny.yml [(#629)](419b1cb)
* chore: split vc gen on some methods to migrate to Dafny 4.4 [(#627)](fdc65ca)
* test: restore CODEOWNERS and daily CI [(#624)](ff823ac)
* chore: update template to point to public repo [(#626)](2b07a39)
* chore: remove unused release step in test-prod [(#623)](9883933)
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