Permalink
Browse files

Various bugfixes and new TODO items

  • Loading branch information...
smfreegard committed Jan 18, 2012
1 parent 032b8be commit fc072a0059c97c6c1254db967d3a299d8ad810ae
Showing with 8 additions and 2 deletions.
  1. +3 −0 TODO
  2. +2 −1 plugins/dnswl.js
  3. +2 −0 plugins/early_talker.js
  4. +1 −1 transaction.js
View
3 TODO
@@ -4,3 +4,6 @@
- Ability to modify the body of email (e.g add a banner) and have it re-write the data_lines
- Create a config file for each of the core shipping configs, so people have something as a baseline
- IMAP server (long shot for now)
+- Spool data to disk once databytes reaches a certain size to save memory
+- Add the option to define special/important plugins that are always run, even if an earlier
+ plugin on the same hook returns OK
View
@@ -23,7 +23,8 @@ exports.register = function() {
}
var self = this;
- ['ehlo','helo','mail','rcpt'].forEach(function (hook) {
+ // IMPORTANT: don't run this on hook_rcpt otherwise we're an open relay...
+ ['ehlo','helo','mail'].forEach(function (hook) {
self.register_hook(hook, 'check_dnswl');
});
}
View
@@ -5,6 +5,8 @@ exports.register = function() {
};
exports.check_early_talker = function(next, connection) {
+ // Don't delay AUTH/RELAY clients
+ if (connection.relaying) return next();
var pause = this.config.get('early_talker.pause');
if (pause) {
setTimeout(function () { _check_early_talker(connection, next) }, pause);
View
@@ -54,7 +54,7 @@ Transaction.prototype.end_data = function() {
Transaction.prototype.add_header = function(key, value) {
this.header.add(key, value);
- this.reset_headers();
+ if (this.header_pos > 0) this.reset_headers();
};
Transaction.prototype.reset_headers = function () {

0 comments on commit fc072a0

Please sign in to comment.