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 explicit missing CA file error #135

Merged
merged 5 commits into from
Nov 13, 2024
Merged

Add explicit missing CA file error #135

merged 5 commits into from
Nov 13, 2024

Conversation

wojcik91
Copy link
Contributor

Make it clear when reading specified CA file fails.
Avoid blanket conversion of all std::io errors and add a dedicated error variant for command execution failure.

@wojcik91 wojcik91 self-assigned this Nov 13, 2024
@wojcik91 wojcik91 merged commit 3aaaf60 into dev Nov 13, 2024
1 check passed
@wojcik91 wojcik91 deleted the missing_ca_file_error branch November 13, 2024 14:08
t-aleksander added a commit that referenced this pull request Nov 19, 2024
* CI: fix re-creating manifests

* chore: log version with git commit hash on startup (#122)

* Bump defguard_wireguard_rs to v0.4.8

* Adopt new WGApi (#124)

* Nix flake

* Enable building ARMv7 images (#125)

* build 32bit ARM images

* enable workflow on PRs for testing

* fix typo

* fix invalid tag format

* remove test trigger

---------

Co-authored-by: Maciej Wójcik <wojcik91@gmail.com>

* Make a pre-release and release docker build workflow (#126)

* add flavor to the docker meta action

* add iptables and ebtables to the docker image (#129)

* Restart gateway after updating it (deb and rpm) (#130)

* add after install script

* remove sudo

* bump version to 1.0.0 (#131)

* Add explicit missing CA file error (#135)

* update gitignore

* add path options typing

* update dev flake

* make io-related errors clearer

* fix log content

---------

Co-authored-by: Maciej Wójcik <maciek@wjck.pl>

* bump version

---------

Co-authored-by: Adam Ciarciński <aciarcinski@teonite.com>
Co-authored-by: Jacek Chmielewski <jchmielewski@teonite.com>
Co-authored-by: Maciek <mwojcik@teonite.com>
Co-authored-by: Maciej Wójcik <wojcik91@gmail.com>
Co-authored-by: Maciek <19913370+wojcik91@users.noreply.github.com>
Co-authored-by: Maciej Wójcik <maciek@wjck.pl>
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