v2.8.0: Fix regression with set challenges being ignored #40

Manually merged
coolaj86 5 年之前 將 1 次代碼提交從 dns-challenge-regression-fix合併至 master
所有者

Regression Fix

Backwards compatibility madness.
When you Greenlock.create({ }), challenges will be set by default if
not available. This is good... ish.

When you approveDomains() and set opts.challenges, however, those must
be able to override the defaults.

This was just recently broken and the fix seems to be to make the prior
defaults inaccessible, otherwise it becomes really confusing as to why
a set DNS challenge for local, wild, or private domains is not being
preferred to the (failing) http.

All this crap will be cleaned up in v3...

New Feature

Oh, and this adds support for getZones(), the testing of which brought about the need to fix the regression.

# Regression Fix Backwards compatibility madness. When you Greenlock.create({ }), challenges will be set by default if not available. This is good... ish. When you approveDomains() and set opts.challenges, however, those must be able to override the defaults. This was just recently broken and the fix seems to be to make the prior defaults inaccessible, otherwise it becomes really confusing as to why a set DNS challenge for local, wild, or private domains is not being preferred to the (failing) http. All this crap will be cleaned up in v3... # New Feature Oh, and this adds support for `getZones()`, the testing of which brought about the need to fix the regression.
coolaj86 closed this pull request 5 年之前
coolaj86 changed title from Fix regression with set challenges being ignored to v2.8.0: Fix regression with set challenges being ignored 5 年之前
The pull request has been manually merged as c8b895633c.
登入 才能加入這對話。
No reviewers
未選擇標籤
未選擇里程碑
No Assignees
1 參與者
訊息
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
尚未有任何內容