qdone 2.2.1 → 2.2.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
@@ -94,8 +94,10 @@ function getOptionsWithDefaults(options) {
94
94
  // For API invocations don't force caller to supply default options
95
95
  if (!options)
96
96
  options = {};
97
- // Activate DLQ if any option is set
98
- const dlq = options.dlq || !!(options['dlq-suffix'] || options['dlq-after'] || options['dlq-name'] || options.dlqSuffix || options.dlqAfter || options.dlqName);
97
+ // Activate DLQ if any sub-option is set. Use ?? so that undefined (not
98
+ // passed) falls through to the default, while explicit false is preserved.
99
+ const hasDlqSubOption = !!(options['dlq-suffix'] || options['dlq-after'] || options['dlq-name'] || options.dlqSuffix || options.dlqAfter || options.dlqName);
100
+ const dlq = options.dlq ?? (hasDlqSubOption || undefined);
99
101
  const opt = {
100
102
  // Shared
101
103
  prefix: options.prefix === '' ? options.prefix : (options.prefix || process.env.QDONE_PREFIX || exports.defaults.prefix),
@@ -124,7 +126,7 @@ function getOptionsWithDefaults(options) {
124
126
  delay: options.delay || process.env.QDONE_DELAY || exports.defaults.delay,
125
127
  sendRetries: options.sendRetries || options['send-retries'] || process.env.QDONE_SEND_RETRIES || exports.defaults.sendRetries,
126
128
  failDelay: options.failDelay || options['fail-delay'] || process.env.QDONE_FAIL_DELAY || exports.defaults.failDelay,
127
- dlq: dlq === false ? false : exports.defaults.dlq,
129
+ dlq: dlq ?? exports.defaults.dlq,
128
130
  dlqSuffix: options.dlqSuffix || options['dlq-suffix'] || process.env.QDONE_DLQ_SUFFIX || exports.defaults.dlqSuffix,
129
131
  dlqAfter: options.dlqAfter || options['dlq-after'] || process.env.QDONE_DLQ_AFTER || exports.defaults.dlqAfter,
130
132
  tags: options.tags || undefined,
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "qdone",
3
- "version": "2.2.1",
3
+ "version": "2.2.2",
4
4
  "description": "A distributed scheduler for SQS",
5
5
  "type": "module",
6
6
  "main": "./index.js",
package/src/defaults.js CHANGED
@@ -90,8 +90,10 @@ export function getOptionsWithDefaults (options) {
90
90
  // For API invocations don't force caller to supply default options
91
91
  if (!options) options = {}
92
92
 
93
- // Activate DLQ if any option is set
94
- const dlq = options.dlq || !!(options['dlq-suffix'] || options['dlq-after'] || options['dlq-name'] || options.dlqSuffix || options.dlqAfter || options.dlqName)
93
+ // Activate DLQ if any sub-option is set. Use ?? so that undefined (not
94
+ // passed) falls through to the default, while explicit false is preserved.
95
+ const hasDlqSubOption = !!(options['dlq-suffix'] || options['dlq-after'] || options['dlq-name'] || options.dlqSuffix || options.dlqAfter || options.dlqName)
96
+ const dlq = options.dlq ?? (hasDlqSubOption || undefined)
95
97
 
96
98
  const opt = {
97
99
  // Shared
@@ -123,7 +125,7 @@ export function getOptionsWithDefaults (options) {
123
125
  delay: options.delay || process.env.QDONE_DELAY || defaults.delay,
124
126
  sendRetries: options.sendRetries || options['send-retries'] || process.env.QDONE_SEND_RETRIES || defaults.sendRetries,
125
127
  failDelay: options.failDelay || options['fail-delay'] || process.env.QDONE_FAIL_DELAY || defaults.failDelay,
126
- dlq: dlq === false ? false : defaults.dlq,
128
+ dlq: dlq ?? defaults.dlq,
127
129
  dlqSuffix: options.dlqSuffix || options['dlq-suffix'] || process.env.QDONE_DLQ_SUFFIX || defaults.dlqSuffix,
128
130
  dlqAfter: options.dlqAfter || options['dlq-after'] || process.env.QDONE_DLQ_AFTER || defaults.dlqAfter,
129
131
  tags: options.tags || undefined,