Skip to content

Commit

Permalink
Fix: typos (#415)
Browse files Browse the repository at this point in the history
  • Loading branch information
szepeviktor authored Jul 28, 2024
1 parent 93b532c commit 33c5d28
Show file tree
Hide file tree
Showing 13 changed files with 36 additions and 36 deletions.
8 changes: 4 additions & 4 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@

* **api:** added endpoint to list locked domains ([25102ca](https://github.com/zone-eu/zone-mta/commit/25102ca71010951fb592c6d8bdb4852976a49c1e))
* **loop:** reject messages with too many Received headers ([cbffa6d](https://github.com/zone-eu/zone-mta/commit/cbffa6d3dcadb937b9303a42fbe2603fd1668fb8))
* **mime-head:** Increased allowd MIME node header size to 2MB ([26e4896](https://github.com/zone-eu/zone-mta/commit/26e48967bd96e8aa08b1986513fe4733961bd672))
* **mime-head:** Increased allowed MIME node header size to 2MB ([26e4896](https://github.com/zone-eu/zone-mta/commit/26e48967bd96e8aa08b1986513fe4733961bd672))
* **tls-options:** Retrieve ignoreTLS value from Sending Zone, or default to false ([#365](https://github.com/zone-eu/zone-mta/issues/365)) ([336e835](https://github.com/zone-eu/zone-mta/commit/336e8358be5992bd15392fc07064540fda4aa23d))

## [3.9.0](https://github.com/zone-eu/zone-mta/compare/v3.8.0...v3.9.0) (2024-01-19)
Expand Down Expand Up @@ -122,7 +122,7 @@
### Bug Fixes

* **deploy:** Set up auto publishing to NPM ([#338](https://github.com/zone-eu/zone-mta/issues/338)) ([b1d6fef](https://github.com/zone-eu/zone-mta/commit/b1d6fef8931a4e40042694414318d1dd5fda2914))
* **tests:** Added Github action to run tests ([#341](https://github.com/zone-eu/zone-mta/issues/341)) ([908edf9](https://github.com/zone-eu/zone-mta/commit/908edf9321be5342426127d081997bde116ff23b))
* **tests:** Added GitHub action to run tests ([#341](https://github.com/zone-eu/zone-mta/issues/341)) ([908edf9](https://github.com/zone-eu/zone-mta/commit/908edf9321be5342426127d081997bde116ff23b))

## CHANGELOG

Expand Down Expand Up @@ -193,7 +193,7 @@

- v2.3.2 2020-10-05

- Do not categorize response codes like 550 automaticaly as "recipient"
- Do not categorize response codes like 550 automatically as "recipient"

- v2.3.1 2020-09-24

Expand Down Expand Up @@ -280,7 +280,7 @@

- v1.10.4 2018-08-22

- Fixed an issue with MX connection timeouts where a working MX exisits but never was tried
- Fixed an issue with MX connection timeouts where a working MX exists but never was tried

- v1.10.2 2018-08-16

Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,7 @@ If the message hard bounces (or after too many retries for soft bounces) a bounc

### Blacklist back-off

If the bounce occured because your sending IP is blacklisted then this IP gets disabled for that MX for the next 6 hours and message is retried from a different IP. You can also disable local IP addresses permanently for specific domains with `disabledAddresses` option.
If the bounce occurred because your sending IP is blacklisted then this IP gets disabled for that MX for the next 6 hours and message is retried from a different IP. You can also disable local IP addresses permanently for specific domains with `disabledAddresses` option.

### Error Recovery

Expand Down
2 changes: 1 addition & 1 deletion config/bounces.txt
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@
^550[ \-]+5\.7\.1\s+[\d\.]+ has been blocked,defer,blacklist,Sender IP blacklisted

# 554 5.7.1 Service unavailable; Client host [1.2.3.4] blocked using ee.countries.nerd.dk; Your IP is in ee, rejected based on geographical location
# Should it reject immediatelly, as in most cases we have addresses from the same region?
# Should it reject immediately, as in most cases we have addresses from the same region?
^554[ \-]+5\.7\.1\s+.*rejected based on geographical location,defer,blacklist,Sender IP blacklisted by region

# 421 4.7.1 Service unavailable; Client host [1.2.3.4] blocked using grey.rbl - Rechazo temporal, intentelo mas tarde por favor...Greylisting in action, please try again later
Expand Down
2 changes: 1 addition & 1 deletion config/default.js
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,7 @@ module.exports = {
// is added to the envelope object, so you can also screen messages against some specific attachments.
// This adds some CPU load as attachments need to be decoded and md5 hashes,
// so increase smtpInterfaces.*.processes count to handle the increased load
// Example: 15872511b0d000c239 ATTACHMENT name="foto-02.jpg" type="image/jpeg" size=1922193 md5=6e0a1c5a2276f7afca68ec7ee4c3200c
// Example: 15872511b0d000c239 ATTACHMENT name="photo-02.jpg" type="image/jpeg" size=1922193 md5=6e0a1c5a2276f7afca68ec7ee4c3200c
'core/image-hashes': false, // 'receiver',

// Sign outbound messages with DKIM
Expand Down
8 changes: 4 additions & 4 deletions lib/address-tools.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ let libmime = require('libmime');
module.exports = {
convertAddresses,
parseAddressList,
parseAddressses,
parseAddresses,
normalizeDomain,
normalizeAddress,
flatten,
Expand All @@ -29,7 +29,7 @@ function validateAddress(headers, key) {
set() {
let address = [].concat([...arguments]);
let values = [];
parseAddressses([].concat(address || []), true).forEach(parsed => {
parseAddresses([].concat(address || []), true).forEach(parsed => {
if (!parsed || !parsed.address) {
return;
}
Expand Down Expand Up @@ -74,13 +74,13 @@ function convertAddresses(addresses, withNames, addressList) {
}

function parseAddressList(headers, key, withNames) {
return parseAddressses(
return parseAddresses(
headers.getDecoded(key).map(header => header.value),
withNames
);
}

function parseAddressses(headerList, withNames) {
function parseAddresses(headerList, withNames) {
let map = convertAddresses(
headerList.map(address => {
if (typeof address === 'string') {
Expand Down
2 changes: 1 addition & 1 deletion lib/dkim-relaxed-body.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ class DkimRelaxedBody extends Transform {
let c = chunk[i];

if (state === 'file' && (c === 0x0a || c === 0x0d)) {
// do nothing, found \n or \r at the end of chunk, stil end of file
// do nothing, found \n or \r at the end of chunk, still end of file
} else if (state === 'file' && (c === 0x09 || c === 0x20)) {
// switch to line ending mode, this is the last non-empty line
state = 'line';
Expand Down
18 changes: 9 additions & 9 deletions lib/mail-queue.js
Original file line number Diff line number Diff line change
Expand Up @@ -589,7 +589,7 @@ class MailQueue {
query.seq = seq;
}

// every delete opration gets an unique ID, so if we fail, we can release locked deliveries
// every delete operation gets an unique ID, so if we fail, we can release locked deliveries
let deleteId = 'delete:' + crypto.randomBytes(10).toString('base64');

let collection = this.mongodb.collection(this.options.collection);
Expand Down Expand Up @@ -996,7 +996,7 @@ class MailQueue {
_id: {
$lte: releaseObjectId
},
// skip messages that are currenlty being processed
// skip messages that are currently being processed
locked: false
},
{
Expand Down Expand Up @@ -1082,12 +1082,12 @@ class MailQueue {
});
}

queuCounterUpdate() {
clearTimeout(this.queuCounterTimer);
queueCounterUpdate() {
clearTimeout(this.queueCounterTimer);

let next = () => {
this.queuCounterTimer = setTimeout(() => this.queuCounterUpdate(), 10 * 1000);
this.queuCounterTimer.unref();
this.queueCounterTimer = setTimeout(() => this.queueCounterUpdate(), 10 * 1000);
this.queueCounterTimer.unref();
};

// probably should find a better way than enumerating all cache keys
Expand Down Expand Up @@ -1154,8 +1154,8 @@ class MailQueue {
this.garbageTimer = setTimeout(() => this.checkGarbage(), 60 * 1000);
this.garbageTimer.unref();

this.queuCounterTimer = setTimeout(() => this.queuCounterUpdate(), 10 * 1000);
this.queuCounterTimer.unref();
this.queueCounterTimer = setTimeout(() => this.queueCounterUpdate(), 10 * 1000);
this.queueCounterTimer.unref();
}

/**
Expand Down Expand Up @@ -1274,7 +1274,7 @@ class MailQueue {
}

/**
* Start periodic tasks (garbage colletion and retrieveing deferred elements)
* Start periodic tasks (garbage collection and retrieveing deferred elements)
*
* @param {Function} callback Run once everything is started
*/
Expand Down
2 changes: 1 addition & 1 deletion lib/queue-server.js
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ class QueueServer {
});

client.onData = (data, next) => {
setImmediate(next); // release immediatelly
setImmediate(next); // release immediately

if (!client) {
// client already errored or closed
Expand Down
2 changes: 1 addition & 1 deletion lib/receiver/smtp-proxy.js
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ class SMTPProxy {
child.on('close', (code, signal) => {
this.children.delete(child);
if (!this.closing) {
log.error('SMTP/' + this.name + '/' + pid, 'Reciver process %s for %s exited with %s', childId, this.name, code || signal);
log.error('SMTP/' + this.name + '/' + pid, 'Receiver process %s for %s exited with %s', childId, this.name, code || signal);

// Respawn after 5 seconds
setTimeout(() => this.spawnReceiver(), 5 * 1000).unref();
Expand Down
10 changes: 5 additions & 5 deletions lib/sender.js
Original file line number Diff line number Diff line change
Expand Up @@ -254,10 +254,10 @@ class Sender extends EventEmitter {

// prepare received header, we need this when sending out the message or when sending a bounce with
// message headers contained. we do not modify the stored delivery object, only the current instance in memory
let recivedHeader = Buffer.from(
let receivedHeader = Buffer.from(
this.zone.generateReceivedHeader(delivery, delivery.localHostname || (connection && connection.options.name) || os.hostname())
);
delivery.headers.addFormatted('Received', recivedHeader, 0);
delivery.headers.addFormatted('Received', receivedHeader, 0);

if (err) {
return handleError(delivery, connection, err);
Expand Down Expand Up @@ -344,12 +344,12 @@ class Sender extends EventEmitter {
}

if (config.dkim.enabled) {
// tro to sign the message, this would prepend a DKIM-Signature header to the message
// to to sign the message, this would prepend a DKIM-Signature header to the message
this.signMessage(delivery);
}

let messageHeaders = delivery.headers.build();
let messageSize = recivedHeader.length + messageHeaders.length + delivery.bodySize; // required for SIZE argument
let messageSize = receivedHeader.length + messageHeaders.length + delivery.bodySize; // required for SIZE argument
let messageFetch = this.queue.retrieve(delivery.id);
let messageHash = new StreamHash({
algo: 'md5'
Expand Down Expand Up @@ -1216,7 +1216,7 @@ class Sender extends EventEmitter {
}
}

err.response = err.response || `Nework error: Error connecting to ${mx.host}. ${err.message}`;
err.response = err.response || `Network error: Error connecting to ${mx.host}. ${err.message}`;
err.category = err.category || 'network';
err.logtrail = logtrail;
if (typeof err.temporary !== 'boolean') {
Expand Down
2 changes: 1 addition & 1 deletion lib/sending-zone.js
Original file line number Diff line number Diff line change
Expand Up @@ -457,7 +457,7 @@ class DomainConfig {
// return clone
return JSON.parse(JSON.stringify(this.defaults[key]));
}
// return defualt value
// return default value
return this.defaults[key];
}
return this.domains.get(domain).get(key);
Expand Down
2 changes: 1 addition & 1 deletion lib/smtp-interface.js
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,7 @@ class SMTPInterface {
ca,
cert,

// Socket timeout is set to 10 minutes. This is needed to give enought time
// Socket timeout is set to 10 minutes. This is needed to give enough time
// for the server to process large recipients lists
socketTimeout: this.options.socketTimeout || 10 * 60 * 1000,

Expand Down
12 changes: 6 additions & 6 deletions test/address-tools-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,16 +3,16 @@
let addressTools = require('../lib/address-tools');
let Headers = require('mailsplit').Headers;

module.exports['#parseAddressses, no names'] = test => {
let parsed = addressTools.parseAddressses(['[email protected], <[email protected]>, Andris3 <[email protected]>', 'Andris4 <[email protected]>']);
module.exports['#parseAddresses, no names'] = test => {
let parsed = addressTools.parseAddresses(['[email protected], <[email protected]>, Andris3 <[email protected]>', 'Andris4 <[email protected]>']);

test.deepEqual(['[email protected]', '[email protected]', '[email protected]'], parsed);

test.done();
};

module.exports['#parseAddressses, with names'] = test => {
let parsed = addressTools.parseAddressses(['[email protected], <[email protected]>, Andris3 <[email protected]>', 'Andris4 <[email protected]>'], true);
module.exports['#parseAddresses, with names'] = test => {
let parsed = addressTools.parseAddresses(['[email protected], <[email protected]>, Andris3 <[email protected]>', 'Andris4 <[email protected]>'], true);

test.deepEqual(
[
Expand All @@ -35,8 +35,8 @@ module.exports['#parseAddressses, with names'] = test => {
test.done();
};

module.exports['#parseAddressses, group with names'] = test => {
let parsed = addressTools.parseAddressses(
module.exports['#parseAddresses, group with names'] = test => {
let parsed = addressTools.parseAddresses(
['[email protected], Disclosed:<[email protected]>, Andris3 <[email protected]>;', 'Andris4 <[email protected]>'],
true
);
Expand Down

0 comments on commit 33c5d28

Please sign in to comment.