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

Build and Runtime Stabilizations #136

Merged
merged 1 commit into from
Sep 6, 2023
Merged

Build and Runtime Stabilizations #136

merged 1 commit into from
Sep 6, 2023

Conversation

schrieveslaach
Copy link
Contributor

This PR ensures that the Docker build uses a coherent Debian version across all Docker build stages (Debian Bookworm will be used as a runtime basis and Rust build basis). Additionally, the Rust backend build uses the native-tls crate instead of the OpenSSL crate which should increase development compatibility across different operating systems.

Last but not least, the parsing of host-meta.json resources ignores malformed datetime values and which ensures that other information will be shown in PREvant's UI (e.g. link to OpenAPI specification).

This commit ensures that the Docker build uses a coherent Debian version
across all Docker build stages (Debian Bookworm will be used as a
runtime basis and Rust build basis). Additionally, the Rust backend
build uses the native-tls crate instead of the OpenSSL crate which
should increase development compatibility across different operating
systems.

Last but not least, the parsing of host-meta.json resources ignores
malformed datetime values and which ensures that other information will
be shown in PREvant's UI (e.g. link to OpenAPI specification).
Copy link
Contributor

@samuchila samuchila left a comment

Choose a reason for hiding this comment

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

The checks are successful with no SSL errors. Additionally, on local k3d cluster the same SSL error was reported, but on a fresh build of this branch the error is resolved.

@schrieveslaach schrieveslaach merged commit e079ba5 into master Sep 6, 2023
2 checks passed
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