Skip to content

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.

@imabhichow imabhichow requested a review from a team as a code owner February 9, 2024 23:02
- 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
@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