Merge pull request #6 from harness-community/CI-20805

fix[CI-20805]: use WithTLSPolicy instead of WithTLSPortPolicy to preserve user-configured SMTP port

DOCS.md

@@ -73,6 +73,55 @@ drone secret add \
 
 See [Secret Guide](https://docs.drone.io/secret/) for additional information on secrets.
 
+### Harness CI Plugin Step
+
+When using this plugin in Harness CI pipelines, you can configure it as a Plugin step. The plugin automatically uses DRONE_* environment variables provided by the Harness CI environment for build context (repository, commit, build status, etc.).
+
+```yaml
+pipeline:
+  orgIdentifier: default
+  tags: {}
+  stages:
+    - stage:
+        name: Run Plugin
+        identifier: Email_Notification
+        description: ""
+        type: CI
+        spec:
+          cloneCodebase: false
+          platform:
+            os: Windows
+            arch: Amd64
+          runtime:
+            type: Cloud
+            spec: {}
+          execution:
+            steps:
+              - step:
+                  type: Plugin
+                  name: Email Plugin
+                  identifier: Email_Plugin
+                  spec:
+                    connectorRef: emailtestdocker
+                    image: plugins/email:3.1.0-debug
+                    settings:
+                      from.address: from@example.com
+                      from.name: Magic Elves
+                      host: sandbox.smtp.mailtrap.io
+                      username: <+secrets.getValue("smtp_username")>
+                      password: <+secrets.getValue("smtp_password")>
+                      recipients: user@example.com
+                      port: "2525"
+          caching:
+            enabled: false
+            paths: []
+          buildIntelligence:
+            enabled: false
+  projectIdentifier: example_project
+  identifier: email_notification_pipeline
+  name: Email Notification Pipeline
+```
+
 ### Custom Templates
 
 In some cases you may want to customize the look and feel of the email message

defaults.go

@@ -191,7 +191,7 @@ const DefaultTemplate = `
           <div class="content">
             <table class="main" width="100%" cellpadding="0" cellspacing="0">
               <tr>
-                {{#success build.status}}
+                {{#equal build.status "success"}}
                   <td class="alert alert-good">
                     <a href="{{ build.link }}">
                       Successful build #{{ build.number }}
@@ -203,7 +203,7 @@ const DefaultTemplate = `
                       Failed build #{{ build.number }}
                     </a>
                   </td>
-                {{/success}}
+                {{/equal}}
               </tr>
               <tr>
                 <td class="content-wrap">

docker/Dockerfile.linux.arm64

@@ -3,7 +3,7 @@ FROM golang:1.24-alpine as builder
 WORKDIR /go/src/drone-email
 COPY . .
 
-RUN GOOS=linux GOARCH=arm GOARM=7 CGO_ENABLED=0 go build
+RUN GOOS=linux GOARCH=arm64 CGO_ENABLED=0 go build
 
 FROM alpine:3.20
 

main.go

@@ -394,8 +394,8 @@ func run(c *cli.Context) error {
 		Job: Job{
 			Status:   c.String("job.status"),
 			ExitCode: c.Int("job.exitCode"),
-			Started:  c.Int64("job.started"),
+			Started:  float64(c.Int64("job.started")),
-			Finished: c.Int64("job.finished"),
+			Finished: float64(c.Int64("job.finished")),
 		},
 		Yaml: Yaml{
 			Signed:   c.Bool("yaml.signed"),

plugin.go

@@ -72,8 +72,8 @@ type (
 	Job struct {
 		Status   string
 		ExitCode int
-		Started  int64
+		Started  float64
-		Finished int64
+		Finished float64
 	}
 
 	Yaml struct {
@@ -185,10 +185,13 @@ func (p Plugin) Exec() error {
 	}
 
 	// Handle STARTTLS policy
+	// Note: Use WithTLSPolicy (not WithTLSPortPolicy) to avoid overriding
+	// the user-configured port. WithTLSPortPolicy treats port 25 as "default/unset"
+	// and silently changes it to 587 for TLSOpportunistic/TLSMandatory.
 	if p.Config.NoStartTLS {
-		options = append(options, mail.WithTLSPortPolicy(mail.NoTLS))
+		options = append(options, mail.WithTLSPolicy(mail.NoTLS))
 	} else {
-		options = append(options, mail.WithTLSPortPolicy(mail.TLSOpportunistic))
+		options = append(options, mail.WithTLSPolicy(mail.TLSOpportunistic))
 	}
 
 	client, err := mail.NewClient(p.Config.Host, options...)