Skip to content

feat: Add SmtpHealthCheck - #396

Merged
spencewenski merged 3 commits into
mainfrom
smtp-health-check
Oct 9, 2024
Merged

feat: Add SmtpHealthCheck#396
spencewenski merged 3 commits into
mainfrom
smtp-health-check

Conversation

@spencewenski

Copy link
Copy Markdown
Member

No description provided.

@codecov

codecov Bot commented Oct 9, 2024

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 35.00000% with 26 lines in your changes missing coverage. Please review.

Project coverage is 42.64%. Comparing base (c880805) to head (2a4ef33).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/api/core/health.rs 0.00% 22 Missing ⚠️
src/health_check/email/smtp.rs 71.42% 4 Missing ⚠️
Files with missing lines Coverage Δ
src/config/health_check/mod.rs 100.00% <ø> (ø)
src/health_check/default.rs 100.00% <100.00%> (ø)
src/health_check/mod.rs 10.00% <ø> (ø)
src/health_check/email/smtp.rs 71.42% <71.42%> (ø)
src/api/core/health.rs 0.00% <0.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c880805...2a4ef33. Read the comment docs.

@spencewenski spencewenski mentioned this pull request Oct 9, 2024
@spencewenski
spencewenski merged commit c860236 into main Oct 9, 2024
@spencewenski
spencewenski deleted the smtp-health-check branch October 9, 2024 20:42
@github-actions github-actions Bot mentioned this pull request Oct 9, 2024
spencewenski added a commit that referenced this pull request Oct 9, 2024
## 🤖 New release
* `roadster`: 0.6.4 -> 0.6.5 (✓ API compatible changes)

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

##
[0.6.5](roadster-v0.6.4...roadster-v0.6.5)
- 2024-10-09

### Added

- Add `SmtpHealthCheck`
([#396](#396))
- Allow specifying the smtp port via config
([#395](#395))
- Add smtp client to `AppContext`
([#391](#391))
- Add support to config for email via SMTP (`email-smtp` feature)
([#388](#388))

### Fixed

- Fix config value used for timeout of health check in api and cli
([#397](#397))

### Other

- Add example of sending email using lettre smtp client
([#394](#394))
- Add doc comment explaining how NormalizePathLayer works
([#393](#393))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/MarcoIeni/release-plz/).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Spencer Ferris <3319370+spencewenski@users.noreply.github.com>
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