Merge pull request dbeaver#1126 from dbeaver/hotfix/CB-2558-log-viewer

CB-2558 fix(plugin-log-viewer): Start logs polling on mount

Author: mctep

webapp/packages/plugin-log-viewer/src/LogViewer/LogViewerService.ts

@@ -63,9 +63,9 @@ export class LogViewerService {
 
   toggle(): void {
     if (this.isActive) {
-      this.stopLog();
+      this.settings.active = false;
     } else {
-      this.startLog();
+      this.settings.active = true;
     }
   }
 
@@ -74,14 +74,14 @@ export class LogViewerService {
   }
 
   async startLog(): Promise<void> {
-    if (this.isActive) {
+    if (this.timeoutTaskId !== null) {
       return;
     }
+
     if (!this.isLogViewerAvailable()) {
       throw new Error('Access denied');
     }
     this.failedRequestsCount = 0;
-    this.settings.active = true;
 
     const refreshInterval = this.logViewerSettingsService.settings.isValueDefault('refreshTimeout')
       ? this.coreSettingsService.settings.getValue('app.logViewer.refreshTimeout')
@@ -100,7 +100,6 @@ export class LogViewerService {
       clearTimeout(this.timeoutTaskId);
       this.timeoutTaskId = null;
     }
-    this.settings.active = false;
   }
 
   clearLog(): void {

webapp/packages/plugin-log-viewer/src/LogViewer/useLogViewer.tsx

@@ -7,6 +7,7 @@
  */
 
 import { observable } from 'mobx';
+import { useEffect } from 'react';
 
 import { useObjectRef, useObservableRef } from '@cloudbeaver/core-blocks';
 import { useService } from '@cloudbeaver/core-di';
@@ -22,6 +23,13 @@ interface Props {
 export function useLogViewer() {
   const logViewerService = useService(LogViewerService);
 
+  useEffect(() => {
+    logViewerService.startLog();
+    return () => {
+      logViewerService.stopLog();
+    };
+  }, []);
+
   const props: Props = useObservableRef(
     () => ({ selectedItem: null }),
     { selectedItem: observable.ref },