Merge Currrent Go Migration to Main#5
Merged
Merged
Conversation
…d the syslog pipeline to use servers logger.
- Updated the configuration structure to include server modes and self-logging levels. - Introduced mutex for thread-safe access to global configuration. - Implemented functions to load and save configuration from/to JSON files. - Added a new parser package with a placeholder for parser processing logic. - Enhanced the pipeline package to manage node states and processing more effectively. - Removed the old server implementation and replaced it with processor functions for TCP and UDP servers. - Deleted unused syslog parsing logic and utility functions. - Introduced a writer package with a placeholder for writer processing logic.
- Changed base image in Dockerfile to golang:1.25.1-bookworm. - Added installation of build-essential and ca-certificates in Dockerfile. - Updated environment variables for server configuration in main.go. - Enhanced configuration generation logic to include SELF_LOGGING_LEVEL. - Enabled default file writer in writer.go. - Cleaned up docker-compose files for better clarity and consistency. - Introduced docker-compose-dev.yml for development setup.
…vers for the time being.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request migrates the
simply_syslogproject from Python to Go, significantly refactoring the codebase, build, and deployment processes. It introduces a new Go-based server implementation, updates Docker and development tooling, and revises the documentation to reflect these changes. The most important changes are grouped below by theme.Migration to Go & Codebase Refactor:
cmd/simplysyslog/main.go), introducing a modular pipeline architecture and support for UDP, with TCP and TLS in progress.config/config.json,config/server_logging_config.json). [1] [2]Build & Deployment Updates:
Dockerfileto build and run the Go application, introduces multi-stage builds, and updates environment variables for configuration.docker-compose-dev.ymlfor easier local testing of the Go-based server..dockerignoreto exclude development and configuration files from Docker builds..idea/.gitignoreentries for workspace-specific files.Documentation:
README.mdto describe the Go implementation, new build and deployment instructions, and revised configuration options. Clarifies the status of TCP/TLS features and environment-based configuration. [1] [2] [3] [4]These changes modernize the project, improve performance and maintainability, and set the stage for future feature development.
Note: This PR description was generated via github copilot, I have manually reviewed it to be accurate.