fix(dcrouter): remove legacy handling of emailConfig.routes that added domain-based routes
This commit is contained in:
@@ -1,5 +1,13 @@
|
|||||||
# Changelog
|
# Changelog
|
||||||
|
|
||||||
|
## 2026-02-12 - 5.0.5 - fix(dcrouter)
|
||||||
|
remove legacy handling of emailConfig.routes that added domain-based routes
|
||||||
|
|
||||||
|
- Removed loop that added domain-based email routes from emailConfig.routes into emailRoutes
|
||||||
|
- Previously created match.domains by extracting the recipient domain (split on '@') and defaulted forward target port to 25
|
||||||
|
- Removed creation of TLS passthrough configuration for those forwarded routes
|
||||||
|
- This prevents duplicate or incorrect domain-based routes being appended during email route construction
|
||||||
|
|
||||||
## 2026-02-12 - 5.0.4 - fix(cache)
|
## 2026-02-12 - 5.0.4 - fix(cache)
|
||||||
use user-writable ~/.serve.zone/dcrouter for TsmDB and centralize data path logic
|
use user-writable ~/.serve.zone/dcrouter for TsmDB and centralize data path logic
|
||||||
|
|
||||||
|
|||||||
@@ -3,6 +3,6 @@
|
|||||||
*/
|
*/
|
||||||
export const commitinfo = {
|
export const commitinfo = {
|
||||||
name: '@serve.zone/dcrouter',
|
name: '@serve.zone/dcrouter',
|
||||||
version: '5.0.4',
|
version: '5.0.5',
|
||||||
description: 'A multifaceted routing service handling mail and SMS delivery functions.'
|
description: 'A multifaceted routing service handling mail and SMS delivery functions.'
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -568,29 +568,6 @@ export class DcRouter {
|
|||||||
emailRoutes.push(routeConfig);
|
emailRoutes.push(routeConfig);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Add email domain-based routes if configured
|
|
||||||
if (emailConfig.routes) {
|
|
||||||
for (const route of emailConfig.routes) {
|
|
||||||
emailRoutes.push({
|
|
||||||
name: route.name,
|
|
||||||
match: {
|
|
||||||
ports: emailConfig.ports,
|
|
||||||
domains: route.match.recipients ? [route.match.recipients.toString().split('@')[1]] : []
|
|
||||||
},
|
|
||||||
action: {
|
|
||||||
type: 'forward',
|
|
||||||
targets: route.action.type === 'forward' && route.action.forward ? [{
|
|
||||||
host: route.action.forward.host,
|
|
||||||
port: route.action.forward.port || 25
|
|
||||||
}] : undefined,
|
|
||||||
tls: {
|
|
||||||
mode: 'passthrough'
|
|
||||||
}
|
|
||||||
}
|
|
||||||
});
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
return emailRoutes;
|
return emailRoutes;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -3,6 +3,6 @@
|
|||||||
*/
|
*/
|
||||||
export const commitinfo = {
|
export const commitinfo = {
|
||||||
name: '@serve.zone/dcrouter',
|
name: '@serve.zone/dcrouter',
|
||||||
version: '5.0.4',
|
version: '5.0.5',
|
||||||
description: 'A multifaceted routing service handling mail and SMS delivery functions.'
|
description: 'A multifaceted routing service handling mail and SMS delivery functions.'
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user