Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CMKC] Email Audits for Review #1724

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

[CMKC] Email Audits for Review #1724

wants to merge 3 commits into from

Conversation

jessewoo
Copy link
Contributor

Source JIRA card(s) and hubzero ticket(s)

https://sdx-sdsc.atlassian.net/browse/USP-193

Brief summary of the issue

PART 2: Audits the different email setTo() of other files
PART 1: #1723

Brief summary of the fix

Did a grep on other files on core changes, in the email object for those files, updated to be sent Bcc (blind carbon copy) instead of To or Cc.

Brief summary of your testing

Have not tried testing on woo.aws.hubzero.org

Do the change needs to be hotfixed to any production hubs before a normal core rollout

Not at the moment, we can do this when there is more time.

Double check someone is assigned to review the ticket

Yes - Nick and David

@jessewoo
Copy link
Contributor Author

able to update the files from grep:

image

@jessewoo
Copy link
Contributor Author

image

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