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.

Sign up for GitHub

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

Jump to bottom

Handle and log misconfiguration #20

Closed
34N0 opened this issue Jan 6, 2024 · 1 comment · Fixed by #46
Closed

Handle and log misconfiguration #20

34N0 opened this issue Jan 6, 2024 · 1 comment · Fixed by #46
Labels
enhancement New feature or request low priority

Comments

@34N0
Copy link
Owner

34N0 commented Jan 6, 2024

Currently failed config file parsing is not handled and logged. There are no crashes, but the default values are used without any explanation.
@34N0 34N0 added the enhancement New feature or request label Jan 6, 2024
@34N0 34N0 modified the milestone: release Jan 6, 2024
@34N0
Copy link
Owner Author

34N0 commented Jan 12, 2024

strip this from release milestone, since it doesn't cause crashes & configuration is simple

34N0 added a commit that referenced this issue Jan 21, 2024
@34N0
refactor: update deps, more concise config, better logging
17a1637 
fix #43 #42 #20
@34N0 34N0 mentioned this issue Jan 21, 2024
Merged
@34N0 34N0 closed this as completed in #46 Jan 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request low priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor: update deps, more concise config, better logging 34N0/pam-authramp
1 participant
@34N0