Skip to content

fix: improve logging in task execution#44

Open
alvarobernal2412 wants to merge 2 commits into
developfrom
fix-status-issues
Open

fix: improve logging in task execution#44
alvarobernal2412 wants to merge 2 commits into
developfrom
fix-status-issues

Conversation

@alvarobernal2412

Copy link
Copy Markdown
Collaborator

This pull request introduces several enhancements and fixes across multiple files, focusing on improving error handling, logging, task management, and testing. Key changes include more robust error handling, detailed logging for debugging, atomic task updates, and refined test assertions.

Error Handling Improvements:

Logging Enhancements:

Task Management Improvements:

  • controllers/filemanager/index.js: Refined updateTask to ensure atomic updates by writing new files first and removing outdated files only after successful writes.

Testing Adjustments:

  • tests/test.js: Modified test assertions to use assert.notStrictEqual instead of assert.strictEqual for verifying API responses, ensuring better validation of response differences. [1] [2]

Configuration Updates:

  • .env.example: Added new environment variables OASTLM_MODULE_DISABLED and GOV_LOG_LEVEL for module control and logging configuration.

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.

Task generates logs every ~3 seconds regardless of defined interval PUT method allows update without id in body, leading to unmanageable records

1 participant