[pull] master from postalsys:master#174
Merged
pull[bot] merged 5 commits intoweiyilai:masterfrom Feb 5, 2026
Merged
Conversation
Extract duplicated logic into shared helpers:
- resolveLabels() in gmail-client for label ID resolution
- throwAsBoom() in routes-ui for export error handling
- Shared jobOpts/removePolicy in webhooks pushToQueue
- Shared retention variable in base-client queueMessage
- Test helpers (createCallSimulator, atomicCheckAndAdd, etc.)
Replace verbose patterns with idiomatic equivalents:
- Boom.badRequest() instead of Boom.boomify(new Error(), {statusCode: 400})
- if/else chain instead of nested ternary in outlook-client $select
- Consolidate redundant null checks in imap-client
Remove redundant comments that restate what the code does,
hoist top-level imports, and consolidate repetitive test cases.
…ssages Add ENOAUTH and EOAUTH2 to SMTP error descriptions so OAuth2 failures produce proper status objects. Discard non-retryable errors (EAUTH, ENOAUTH, EOAUTH2, ETLS, EENVELOPE, EMESSAGE, EPROTOCOL) immediately in the submit worker instead of retrying up to 10 times. Extend SMTP verification switch statements with human-readable messages for six additional error codes.
Bump nodemailer to 8.0.0 and undici to 7.20.0. Add new translatable error messages for improved connection diagnostics. Rename "SMTP server" to "Mail server" in account UI. Update Google crawler IPs and regenerate SBOM/licenses.
Compose descriptive err.message at error creation in OAuth provider classes so that BullMQ failedReason includes provider name, grant type, HTTP status, and provider error response instead of just "Token request failed".
- Throw on null mailboxInfo in getMailboxStatus to prevent TypeError crash - Use append mode when resuming exports to preserve previously exported data - Wrap markInterruptedAsFailed loop in try-catch so one failure does not abandon remaining entries; also clean up exports interrupted during indexing - Add 30s timeout to stream finalization to prevent indefinite hang - Remove encodeURIComponent from Outlook batch URLs to fix OData query parsing - Fix test score calculation to match production algorithm (SHA-256, mod 1M)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )