Skip to content

Fix bug 424#907

Open
peteri14 wants to merge 1 commit intolouislam:masterfrom
peteri14:fix-424
Open

Fix bug 424#907
peteri14 wants to merge 1 commit intolouislam:masterfrom
peteri14:fix-424

Conversation

@peteri14
Copy link
Copy Markdown

@peteri14 peteri14 commented Jan 1, 2026

⚠️⚠️⚠️ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules:
https://github.com/louislam/dockge/blob/master/CONTRIBUTING.md

Tick the checkbox if you understand [x]:

  • I have read and understand the pull request rules.

Description

Fixes #(424) This pull request fixes the issue of bug 424, because that error is generated due to the problem that in those cases the dicrectory does not exist in which the system try to execute the 'docker compose ps --format json' command. A simple check is added, (check that the directory is managed by dockge), before the status is requested.

The other correction I have made is to minimise unnecessary logging messages, so I have converted the console.log message to log.debug. These messages can fill up the logs, since they are generated every 5 seconds as long as we a "viewing" the stack.

Type of change

Please delete any options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • My code follows the style guidelines of this project
  • I ran ESLint and other linters for modified files
  • I have performed a self-review of my own code and tested it
  • I have commented my code, particularly in hard-to-understand areas
    (including JSDoc for methods)
  • My changes generate no new warnings
  • My code needed automated testing. I have added them (this is optional task)

Screenshots (if any)

Please do not use any external image service. Instead, just paste in or drag and drop the image here, and it will be uploaded automatically.

@peteri14 peteri14 mentioned this pull request Jan 1, 2026
2 tasks
krbob added a commit to krbob/dockge that referenced this pull request Apr 3, 2026
Cherry-pick PR louislam#907: check isManagedByDockge before calling
getServiceStatusList to prevent ENOENT crash. Also replace
console.log with log.debug in getComposeOptions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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