Add firewall logging and fix direnv .env loading #15
Conversation
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
Introduces comprehensive logging functionality to the devcontainer firewall system to help developers debug network connectivity problems: - Enhanced init-firewall.sh with configurable logging levels and destinations - Added environment variables for firewall logging configuration - Created detailed documentation for logging setup and usage - Updated examples and README with logging references - Supports syslog and file-based logging with configurable verbosity This feature is disabled by default to maintain performance and only activates when explicitly configured via environment variables.
- Configure direnv with dotenv_if_exists in direnvrc for automatic .env loading - Add .envrc template file to enable safe environment variable loading - Update README with environment variable loading documentation - Ensure proper ownership of direnv configuration files This complements the firewall logging feature by improving environment variable management and making the development environment more seamless.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
This PR introduces two key improvements to the devcontainer:
Features Added
Firewall Logging
FIREWALL_LOG_ENABLED,FIREWALL_LOG_RATE)docs/firewall-logging.mdDirenv Environment Loading
dotenv_if_existsto automatically load .env filesTest Plan
Both features enhance the development experience while maintaining the container's security model.