Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add debug settings #18

Merged
merged 3 commits into from
Feb 21, 2025

Conversation

parchinskiy
Copy link
Contributor

Привет.
LogGrok пишет дампы и это здорово. Но есть проблемы:

  1. Довольно критичная: LogGrok записывает дампы без уведомления и что самое печальное, в не самое очевидное место. У меня какое-то время был системный SSD на 250Гб. В какой-то момент мне стало не хватать памяти и я начал разбираться куда она делась. Я довольно сильно удивился, когда обнаружил, что LogGrok записал дампов на 30Гб в папку локального администратора.

  2. Некритичная: LogGrok запрашивает права администратора при включении записи отладочных дампов. К сожалению, не всегда на устройстве есть доступ к учетной записи с правами администратора. Здорово то, что LogGrok продолжает работать, даже если ему не предоставить эти права, но постоянный UAC с запросом прав администратора немного раздражает.

В этом PR:

  • Добавил настройки для включения/выключения записи отладочных дампов в appsettings.yaml
  • По умолчанию запись дампов выключена
  • Если пользователь не включает запись дампов UAC не появляется
  • Настройка, через которую можно регулировать максимальное количество дампов, на случай если снимать дампы захочется, а памяти на компьютере не очень много

Возможно стоит добавить кастомизацию пути для записи дампов.

Константин Мокраусов added 3 commits February 20, 2025 20:12
@pekabon pekabon merged commit 48fa612 into pekabon:master Feb 21, 2025
2 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants