Skip to content

Add debug settings#18

Merged
pekabon merged 3 commits intopekabon:masterfrom
parchinskiy:add_settings_for_collecting_dumps
Feb 21, 2025
Merged

Add debug settings#18
pekabon merged 3 commits intopekabon:masterfrom
parchinskiy:add_settings_for_collecting_dumps

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants