fix(routing): spurious warning when user DefaultMailbox is set to "Default"#2071
Conversation
Co-authored-by: rnwood <1327895+rnwood@users.noreply.github.com>
|
|
@copilot The build triggered by this PR has failed. Please use the Azure DevOps REST API to investigate and fix the issues. 🚨 IMPORTANT: Only proceed if this is the most recent commit on your branch. If a newer commit exists, ignore this notification. Build Details:
Instructions for @copilot:
The Azure DevOps REST API provides comprehensive build failure information without requiring browser access. |
Investigated the build failure. The failing test is This is a pre-existing CI infrastructure issue — the same failure occurs in other unrelated PRs (e.g., build 5877 for PR #1608). It's caused by a TLS certificate not being available in the Windows CI environment, and is not related to the change in commit eb34989. |
|
@copilot The build triggered by this PR has failed. Please use the Azure DevOps REST API to investigate and fix the issues. 🚨 IMPORTANT: Only proceed if this is the most recent commit on your branch. If a newer commit exists, ignore this notification. Build Details:
Instructions for @copilot:
The Azure DevOps REST API provides comprehensive build failure information without requiring browser access. |



When a user's
DefaultMailboxis configured as"Default"(the implicit fallback), the server logged a warning on every received message because it searched for"Default"in the explicitly configured mailboxes list — where it never appears, since it's auto-created at runtime.Change
In
Smtp4devServer.cs, skip the configured-mailbox lookup whenuserDefaultMailboxequalsMailboxOptions.DEFAULTNAME:The warning
Mailbox 'Default' was not found. Falling back to default mailboxis now only emitted when a user references a genuinely missing non-default mailbox name.Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.