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
dird: show current and allowed console connections #1487
Conversation
ef7b95b
to
f094a2a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks goods, see my remark
Also if you change something maybe you can fix the check-source warning |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Depending on what we want to accomplish this needs to be changed somewhat.
f838044
to
3f1dc90
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't have a good idea how to make this nicer, but the current approach is not great. Too much mixing of concerns and just too complicated in my opinion.
3f1dc90
to
d57545f
Compare
d57545f
to
cb91d80
Compare
cb91d80
to
3f1dc90
Compare
When the allowed number of console connectinos was exceeded, before only the fact that it is exceeded was logged without numbers. Now both the allowed maximum and the current value are printed.
This adds a lease-based system to count open console connections. When the connection limit is exceeded, the default console will log an informational message, only named consoles will be denied access.
3f1dc90
to
7234fb8
Compare
When the allowed number of console connectinos was exceeded, before only the fact that it is exceeded was logged without numbers.
Now both the allowed maximum and the current value are printed.
Thank you for contributing to the Bareos Project!
Please check
If you have any questions or problems, please give a comment in the PR.
Helpful documentation and best practices
Checklist for the reviewer of the PR (will be processed by the Bareos team)
Make sure you check/merge the PR using
devtools/pr-tool
to have some simple automated checks run and a proper changelog record added.General
Source code quality
Tests