Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add the proxy Middleware, enhance logger and config pkgs #109

Merged
merged 6 commits into from
Aug 13, 2024

Conversation

yunkon-kim
Copy link
Member

This PR will introduce significant updates to the proxy middleware to forward specific requests to the subsystem that has the corresponding features.

This could be the base line of #108

Changes are detailed below:

  • Add the proxy Middleware
    • Developed proxy middleware to forward specific requests to target servers
    • Implemented routes to forward certain requests to the Tumblebug API server
    • Enhanced request routing for better performance and flexibility
  • Check the readiness of Tumblebug
    • Added checks to ensure the readiness of the Tumblebug service, enhancing reliability and stability.
  • Add NoOpLogger
    • Introduced a NoOpLogger to prevent unnecessary WARN logs from being printed, streamlining log output.
  • Set route groups
    • Organized and set route groups for better structure and maintenance.
    • Note: The current route package will be deprecated.
  • Set custom caller for consistent log format
    • Implemented a custom caller to ensure log formats remain consistent across different running environments.
  • Update configurations and environment variables
    • Revised configuration files and environment variables for better integration and ease of management.

@yunkon-kim yunkon-kim requested a review from seokho-son as a code owner August 13, 2024 08:51
@yunkon-kim
Copy link
Member Author

/approve

@github-actions github-actions bot added the approved This PR is approved and will be merged soon. label Aug 13, 2024
@cb-github-robot cb-github-robot merged commit 5c63306 into cloud-barista:main Aug 13, 2024
2 checks passed
@yunkon-kim yunkon-kim deleted the 240808-22 branch August 22, 2024 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved This PR is approved and will be merged soon.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants