Skip to content

Add explicit missing CA file error#135

Merged
wojcik91 merged 5 commits intodevfrom
missing_ca_file_error
Nov 13, 2024
Merged

Add explicit missing CA file error#135
wojcik91 merged 5 commits intodevfrom
missing_ca_file_error

Conversation

@wojcik91
Copy link
Copy Markdown
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 requested a review from moubctez November 13, 2024 10:02
@wojcik91 wojcik91 self-assigned this Nov 13, 2024
@wojcik91 wojcik91 merged commit 3aaaf60 into dev Nov 13, 2024
@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