Skip to content

auth: clarify Resend webhook log messages - #209

Merged
aspiers merged 1 commit into
mainfrom
fix/resend-webhook-log-message
Jul 20, 2026
Merged

auth: clarify Resend webhook log messages#209
aspiers merged 1 commit into
mainfrom
fix/resend-webhook-log-message

Conversation

@aspiers

@aspiers aspiers commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Summary

Make Resend webhook events immediately legible in log viewers that emphasize message text over structured properties.

Changes

  • Include the normalized event type and provider in accepted webhook messages.
  • Preserve the existing structured eventType and provider fields.
  • Update tests for both informational and warning-level event messages.
  • Add an operator-facing patch changeset.

Testing

  • pnpm format:check
  • pnpm lint
  • TypeScript project build (tsc --build tsconfig.json)
  • pnpm test — 1,067 tests passed
  • pnpm test:coverage

Notes

Messages now use text such as Received email event delivered from resend.

Summary by CodeRabbit

  • Improvements
    • Resend webhook logs now include the email event type and provider directly in the message, making them easier to understand in log viewers.

Railway and similar log viewers emphasize message text, so the generic webhook message hid the event type and provider unless operators expanded structured properties.\n\nInclude the normalized event type and provider directly in each accepted webhook message while retaining the existing structured fields.\n\nCo-authored-by: OpenAI <noreply@openai.com>
Copilot AI review requested due to automatic review settings July 20, 2026 16:55
@changeset-bot

changeset-bot Bot commented Jul 20, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 0af6d7f

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercel Bot commented Jul 20, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
epds-demo Ready Ready Preview, Comment Jul 20, 2026 4:55pm

Request Review

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@railway-app
railway-app Bot temporarily deployed to ePDS / ePDS-pr-209 July 20, 2026 16:55 Destroyed
@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: db83e936-5c43-4f10-a902-41563792b01d

📥 Commits

Reviewing files that changed from the base of the PR and between 28c3521 and 0af6d7f.

📒 Files selected for processing (3)
  • .changeset/resend-log-message-context.md
  • packages/auth-service/src/__tests__/resend-webhook.test.ts
  • packages/auth-service/src/routes/resend-webhook.ts

📝 Walkthrough

Walkthrough

Resend webhook logs now include the normalized email event type and provider in their human-readable messages. Info and warning log assertions were updated, and a Changesets entry documents the operator-facing change.

Changes

Resend webhook logging

Layer / File(s) Summary
Contextual webhook log messages
packages/auth-service/src/routes/resend-webhook.ts, packages/auth-service/src/__tests__/resend-webhook.test.ts, .changeset/resend-log-message-context.md
Webhook log messages now include the normalized event type and provider, with updated info and warning expectations and operator-facing release metadata.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers: copilot

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title is concise and accurately summarizes the main change: clarifying Resend webhook log messages.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/resend-webhook-log-message

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install timed out. The project may have too many dependencies for the sandbox.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sonarqubecloud

Copy link
Copy Markdown

@coveralls-official

Copy link
Copy Markdown

Coverage Report for CI Build 29761633787

Coverage remained the same at 57.272%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: 1 of 1 lines across 1 file are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 3068
Covered Lines: 1752
Line Coverage: 57.11%
Relevant Branches: 1910
Covered Branches: 1099
Branch Coverage: 57.54%
Branches in Coverage %: Yes
Coverage Strength: 6.06 hits per line

💛 - Coveralls

@railway-app

railway-app Bot commented Jul 20, 2026

Copy link
Copy Markdown

🚅 Deployed to the ePDS-pr-209 environment in ePDS

Service Status Web Updated (UTC)
@certified-app/demo untrusted ✅ Success (View Logs) Web Jul 20, 2026 at 4:57 pm
@certified-app/auth-service ✅ Success (View Logs) Web Jul 20, 2026 at 4:56 pm
@certified-app/demo ✅ Success (View Logs) Web Jul 20, 2026 at 4:56 pm
@certified-app/pds-core ✅ Success (View Logs) Web Jul 20, 2026 at 4:56 pm

@aspiers
aspiers merged commit 1e8257e into main Jul 20, 2026
29 checks passed
@aspiers
aspiers deleted the fix/resend-webhook-log-message branch July 20, 2026 17:43
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.

2 participants