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

feat: Big refactor. #234

Merged
merged 1 commit into from
Feb 4, 2025
Merged

Conversation

alilleybrinker
Copy link
Member

This refactor eliminates the gitoid crate, merging its key logic into the omnibor crate. It also substantially reorganizes the OmniBOR crate. There are too many changes to cover here, many of them minor.

The most essential change is that cryptography providers and the hash algorithm type have been separated from each other. Sha256 is now a marker type that can be produced by any compliant cryptography provider (RustCrypto, OpenSSL, and BoringSSL today), meaning that information about the identity of the cryptography provider doesn't leak into the identity of the resulting ArtifactId or InputManifest.

@alilleybrinker alilleybrinker added type: feature New feature or request crate: gitoid Relating to the gitoid crate crate: omnibor Relating to the omnibor crate crate: omnibor-cli Relating to the omnibor-cli crate type: refactor Regular maintenance of the code labels Feb 4, 2025
@alilleybrinker alilleybrinker self-assigned this Feb 4, 2025
This refactor eliminates the `gitoid` crate, merging its key logic into the
`omnibor` crate. It also substantially reorganizes the OmniBOR crate. There
are too many changes to cover here, many of them minor.

The most essential change is that cryptography providers and the hash algorithm
type have been separated from each other. `Sha256` is now a marker type that
can be produced by any compliant cryptography provider (RustCrypto, OpenSSL,
and BoringSSL today), meaning that information about the identity of the
cryptography provider doesn't leak into the identity of the resulting
`ArtifactId` or `InputManifest`.

Signed-off-by: Andrew Lilley Brinker <[email protected]>
@alilleybrinker alilleybrinker force-pushed the alilleybrinker/gitoid-crate-destroy branch from 067f439 to 063987b Compare February 4, 2025 23:45
@alilleybrinker alilleybrinker merged commit b20eff6 into main Feb 4, 2025
10 checks passed
@alilleybrinker alilleybrinker deleted the alilleybrinker/gitoid-crate-destroy branch February 4, 2025 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crate: gitoid Relating to the gitoid crate crate: omnibor Relating to the omnibor crate crate: omnibor-cli Relating to the omnibor-cli crate type: feature New feature or request type: refactor Regular maintenance of the code
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant