F file read error
Summary
See #30 (closed). Having an empty path for the certificate leads to a vanilla basic iostream error
Focus
The basic exception is caught and changed into a slightly more precise Exception with error code.
Test Environment
e.g. set cert path to empty string and try to connect
Check List for the Author
Please, prepare your MR for a review. Be sure to write a summary and a focus and create gitlab comments for the reviewer. They should guide the reviewer through the changes, explain your changes and also point out open questions. For further good practices have a look at our review guidelines
-
All automated tests pass -
Reference related Issues -
Up-to-date CHANGELOG.md -
Annotations in code (Gitlab comments) - Intent of new code
- Problems with old code
- Why this implementation?
Check List for the Reviewer
-
I understand the intent of this MR -
All automated tests pass -
Up-to-date CHANGELOG.md -
The test environment setup works and the intended behavior is reproducible in the test environment -
In-code documentation and comments are up-to-date. -
Check: Are there specifications? Are they satisfied?
For further good practices have a look at our review guidelines.
Merge request reports
Activity
added 1 commit
- 6aa71775 - MAINT: change vanilla exception to more specific one
assigned to @daniel
added 1 commit
- 1c57270a - WIP TEST: Wrote test which did not fail before.
unassigned @daniel
assigned to @florian
unassigned @henrik
added 110 commits
-
1c57270a...8ef5aeb0 - 109 commits from branch
dev - 986d42f2 - Merge branch 'dev' into f-file-read-error
-
1c57270a...8ef5aeb0 - 109 commits from branch
added 1 commit
- 84773e02 - API: Rename UnkownConnectionError to ConnectionConfigurationError
added 1 commit
- a10656ae - DOC: revert changelog changes (not necessary yet)