Skip to content

Устранение блокировки файла логов при доступе по SMB#33

Closed
maxten wants to merge 1 commit intomainfrom
fix-smb-log-locking
Closed

Устранение блокировки файла логов при доступе по SMB#33
maxten wants to merge 1 commit intomainfrom
fix-smb-log-locking

Conversation

@maxten
Copy link
Copy Markdown

@maxten maxten commented Oct 27, 2025

Устраняем проблему блокировки log файлов при подключении к файловым ресурсам Windows по SMB
Что сделано:

  1. Файл не блокируется, при обновлении или пересоздании файла происходит обновление отображения в DirectumLogViewer
  2. Устранен дублирующий код.

@maxten maxten requested a review from desol8686 October 27, 2025 09:35
Comment thread LogReader/LogWatcher.cs
{
ReadToEndLine();
timer.Start();
try
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

такая конструкция может засыпать пользователя ошибками

var tenant = this.TenantFilter.SelectedValue as string;
var level = this.LevelFilter.SelectedValue as string;
var logger = this.LoggerFilter.SelectedValue as string;
GetCurrentSelectionFilters(out var tenant, out var level, out var logger);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

эти изменения выглядят грязно. почему через out. если рефакторить, то рефакторить через структуру/класс

@desol8686
Copy link
Copy Markdown
Collaborator

Баг починен

@desol8686 desol8686 closed this Apr 15, 2026
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