Skip to content

fix: update logging configuration and correct typos in environment files#44

Merged
jhonatapers merged 4 commits intodevelopfrom
fix/log
Jun 1, 2025
Merged

fix: update logging configuration and correct typos in environment files#44
jhonatapers merged 4 commits intodevelopfrom
fix/log

Conversation

@jhonatapers
Copy link
Copy Markdown
Member

This pull request includes updates to the Dockerfile, YAML configuration files, and Log4j2 configuration to improve consistency, correct typos, and enhance logging functionality. The most important changes are grouped below by theme.

Dockerfile Updates:

  • Switched the base image from eclipse-temurin:21.0.7_6-jdk-alpine to eclipse-temurin:21.0.7_6-jre-alpine to use a runtime environment instead of a development kit.
  • Added creation of /var/log/drive-api directory for logging purposes and adjusted ownership of this directory.

Configuration Fixes:

  • Corrected a typo in application-env.yml, changing reuqest-timeout to request-timeout.
  • Fixed a similar typo in application.yml, updating reuqest-timeout to request-timeout in the spring.mvc.async configuration.

Log4j2 Configuration Enhancements:

  • Updated property names in log4j2-spring.xml to use camelCase for consistency (e.g., ROOT_LOG_LEVEL to rootLogLevelConsole).
  • Changed the root logger and com.callv2 logger levels to all for more detailed logging during development.
  • Updated log file paths and patterns to use /var/log/drive-api for better organization and consistency.

@jhonatapers jhonatapers self-assigned this Jun 1, 2025
@jhonatapers jhonatapers added the bug Something isn't working label Jun 1, 2025
@jhonatapers jhonatapers linked an issue Jun 1, 2025 that may be closed by this pull request
@jhonatapers jhonatapers merged commit 8b401f1 into develop Jun 1, 2025
@github-project-automation github-project-automation Bot moved this to Done in CallV2 Jun 1, 2025
@jhonatapers jhonatapers deleted the fix/log branch June 1, 2025 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[FIX] - Logs Docker

1 participant