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

[FEATURE]-Add timer to debug console operations to automatically disable debug after expiry #113

Open
ndorin opened this issue Aug 20, 2021 · 1 comment
Labels
enhancement New feature or request planning

Comments

@ndorin
Copy link
Contributor

ndorin commented Aug 20, 2021

Is your feature request related to a problem? Please describe.
Currently, setting debug level to > 0 never expires and can cause additional processor overhead if left on unintentionally.

Describe the solution you'd like
Add a timer with a default value whenever the appdebug console command sets the level to > 0. Add an optional argument to the command to explicitly set the timeout to a value different than the default.

This might be a bit tricky to implement with the way the current logic saves the level to a file to persist through reboot. Some thought will have to be put in to consider how to deal with that when the program reboots and restores the previous level.

@ndorin ndorin added the enhancement New feature or request label Aug 20, 2021
@ndorin
Copy link
Contributor Author

ndorin commented Aug 20, 2021

@jtalborough Moved this issue to the PepperDashCore library where the affected code resides

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request planning
Projects
None yet
Development

No branches or pull requests

2 participants