Skip to content

Conversation

@gusfcarvalho
Copy link
Contributor

No description provided.

Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
Copilot AI review requested due to automatic review settings December 30, 2025 22:08
@gusfcarvalho gusfcarvalho enabled auto-merge (squash) December 30, 2025 22:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds environment variable binding for web_base_url and enhances the SES email provider to support formatted "From" addresses with sender names. The web_base_url binding allows the application to read this configuration from the environment, which aligns with the PR title. Additionally, the SES provider now formats sender information as "Name email@domain.com" when a from name is provided, matching the existing SMTP provider functionality.

  • Added web_base_url environment variable binding in the configuration
  • Enhanced SES email provider to support sender name formatting
  • Improved error logging in SES provider to include the from address

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
cmd/root.go Adds binding for web_base_url environment variable to enable configuration from environment
internal/service/email/providers/ses.go Adds support for formatting from addresses with sender names and enhances error logging with from address information

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@gusfcarvalho gusfcarvalho merged commit 0c8f6c0 into main Dec 30, 2025
10 checks passed
@gusfcarvalho gusfcarvalho deleted the gc-fix-web-base-url branch December 30, 2025 22:11
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.

1 participant