Skip to content

feat: allow pass hostname in docker env - #19

Open
tomerqodo wants to merge 2 commits into
cursor_combined_20260121_qodo_grep_cursor_copilot_1_base_feat_allow_pass_hostname_in_docker_env_pr419from
cursor_combined_20260121_qodo_grep_cursor_copilot_1_head_feat_allow_pass_hostname_in_docker_env_pr419
Open

feat: allow pass hostname in docker env#19
tomerqodo wants to merge 2 commits into
cursor_combined_20260121_qodo_grep_cursor_copilot_1_base_feat_allow_pass_hostname_in_docker_env_pr419from
cursor_combined_20260121_qodo_grep_cursor_copilot_1_head_feat_allow_pass_hostname_in_docker_env_pr419

Conversation

@tomerqodo

@tomerqodo tomerqodo commented Jan 21, 2026

Copy link
Copy Markdown

Benchmark PR from qodo-benchmark#419


Note

Adds optional hostname override for SMTP HELO/EHLO and propagates it through configuration and runtime.

  • New SMTP_LOCAL_HOSTNAME in MailConfig and documented in api/.env.example, integration test .env.example, and docker/.env.example
  • docker-compose.yaml passes SMTP_LOCAL_HOSTNAME to the API service
  • SMTPClient.send now sets local_hostname when creating smtplib.SMTP/SMTP_SSL and uses it for EHLO in STARTTLS flow
  • Unit/integration tests updated to expect local_hostname parameter (ANY) in SMTP constructors

Written by Cursor Bugbot for commit 3e5625b. Configure here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants