Dev1#11
Merged
Merged
Conversation
… program will exit
- Add IgnoreTLSErrors option for insecure TLS connections - Add AllowUnsafeHTML option for unsafe HTML embedding in Markdown - Support configuration via config.yaml, environment variables, and CLI flags - Include validation for security-sensitive options - Document security implications in defaults and comments
- Document ignore_tls_errors option with security warnings - Document allow_unsafe_html option with security implications - Add environment variable examples for security options - Include usage guidelines for development vs production
…safe HTML - Add --ignore-tls-errors flag to disable TLS certificate verification - Add --allow-unsafe-html flag to permit unsafe HTML embedding in emails - Integrate with appconfig security settings (config file, env vars, CLI flags) - Add comprehensive help text with security warnings - Display security configuration on startup - Support configuration precedence: CLI flag > env var > config file > default These options are intended for development/testing with self-signed certificates and trusted HTML sources only. Production use should keep defaults.
- BuildEmailMessage: creates email.Message from evidence report - Reads Report.md and converts to HTML via goldmark (GFM + CJK) - Supports both HTML and plain-text content - Generates Message-ID in format: <username#reportid%random@domain> - Attaches first matching screenshot based on evidence.yaml user_agents - Reads From/Reply-To from appconfig - Leaves To/CC/BCC empty for later specification - Validates configuration and report data - Uses github.com/xhit/go-simple-mail/v2 for email transport - Logs warnings when expected files/screenshots are missing
…al/email - Update runEmailDraft to use BuildEmailMessage for message generation - Support HTML and plain-text formats based on configuration - Implement preview display with message details - Prepare for IMAP upload and .eml export (placeholders) - Log security configuration status - Display generated email details including recipients, subject, attachments - Handle includeHTML parameter from --plain-text-only flag
- Changed from email.Email to *smtp.Email for correct type - Fixed type safety for return value - Corrected package reference
- Changed email.Email to *simplemail.Email - Updated displayEmailPreview function signature - Fixed method calls for retrieving email properties - Added simplemail import
…luding markdown to HTML conversion and screenshot attachments.
…ts, supporting HTML/plain-text content and attachments, and add new gitignore entries.
…for better clarity
…htChecks function
…and clean up formatting
…ith error logging
… closures and update user agent string
…hot retrieval logic
- Integrate go-imap/v2 library - Support SSL/STARTTLS/None connection modes - Auto-detect \Drafts mailbox using RFC 6154 SPECIAL-USE - Add --imap-timeout flag with 30s default - Change default behavior to export .eml + upload to IMAP - Add comprehensive documentation
- Add DebugWriter utility for log-level controlled output - Enable IMAP protocol logging in debug mode - Support --verbose alias for --debug
…addresses - Implemented placeholder context for email address expansion based on report ID. - Enhanced email configuration to support placeholders in From and Reply-To fields. - Added validation for email addresses to ensure proper formatting after placeholder expansion. - Updated documentation to reflect changes in email configuration and usage of placeholders.
… unnecessary warnings
- Updated consent settings in app-behavior-config.md for clarity. - Added dependency on btcsuite/btcutil in go.mod. - Enhanced report content structure to include report directory. - Modified report generation to use report directory instead of report ID. - Simplified email address placeholder context to only include report ID. - Updated tests to reflect changes in report directory handling and placeholder expansion. - Removed unused timestamp, date, and time fields from placeholder context and tests.
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.
No description provided.