Skip to content

SLK-87780 | Fix failed sent email when app scope owner is empty#660

Merged
Nashati merged 1 commit intopostee-for-enterprisefrom
hai-slk-87780
Apr 23, 2025
Merged

SLK-87780 | Fix failed sent email when app scope owner is empty#660
Nashati merged 1 commit intopostee-for-enterprisefrom
hai-slk-87780

Conversation

@haiataquasec
Copy link
Copy Markdown
Collaborator

This ticket aims to fix:

  1. Correct expected format of email
  2. Disregard empty string in case server sent (should not happen but you never know)
  3. Use panic recovery in any case of panic error. Though that piece of code should continue the loop and send other emails present in job, it does not perform as expected so I added it just in case.

Related PR in Server: https://bitbucket.org/scalock/server/pull-requests/28142

Resolves: SLK-87780

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Nashati Nashati merged commit 263e682 into postee-for-enterprise Apr 23, 2025
2 of 3 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.

4 participants