Skip to content

Conversation

@Dala0
Copy link
Contributor

@Dala0 Dala0 commented Apr 14, 2025

What does this change?

Causes an e-mail to be sent if a sensitive project is created. Requires data similar to the following be added to the pluto-core configuration file to work as intended: -

play.mailer {
  host = "smtp.a.org"
  port = 25
  ssl = no
  tls = no
  tlsRequired = no
  user = null
  password = null
  debug = no
  timeout = null
  connectiontimeout = null
  mock = no
  props {
  }
}

mail {
  subject = "String"
  sender_name = "String"
  sender_address = "a@a.org"
  recipient_name = "String"
  recipient_address = "b@b.org"
}

How can we measure success?

The e-mail is sent.

This was tested on a machine running macOS 12.6.8 under minikube 1.31.2 and on the dev. system.

@github-actions
Copy link

github-actions bot commented Apr 14, 2025

Unit Test Results

185 tests  ±0   185 ✔️ ±0   45s ⏱️ ±0s
  52 suites ±0       0 💤 ±0 
  52 files   ±0       0 ±0 

Results for commit 787f4ad. ± Comparison against base commit 6bd5da1.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

This was published to a Docker image at AWS_ACCOUNT_ID.dkr.ecr.AWS_REGION.amazonaws.com/pluto-core:1564

@github-actions
Copy link

This was published to a Docker image at AWS_ACCOUNT_ID.dkr.ecr.AWS_REGION.amazonaws.com/pluto-core:1565

@github-actions
Copy link

This was published to a Docker image at AWS_ACCOUNT_ID.dkr.ecr.AWS_REGION.amazonaws.com/pluto-core:1566

@github-actions
Copy link

This was published to a Docker image at AWS_ACCOUNT_ID.dkr.ecr.AWS_REGION.amazonaws.com/pluto-core:1567

Copy link
Contributor

@richpryce richpryce left a comment

Choose a reason for hiding this comment

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

👍🏼

@Dala0 Dala0 merged commit 9ef552d into main Apr 15, 2025
4 checks passed
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.

3 participants