Skip to content

Commit 41e8704

Browse files
build: Enable some modules by default
We don't enable the ECDSA recovery module, because we don't recommend ECDSA recovery for new protocols. In particular, the recovery API is prone to misuse: It invites the caller to forget to check the public key (and the verification function always returns 1). In general, we also don't recommend ordinary ECDSA for new protocols. But disabling the ECDSA functions is not possible because they're not in a module, and let's be honest: disabling ECDSA would mean to ignore reality blatantly.
1 parent 9f8a13d commit 41e8704

File tree

2 files changed

+9
-6
lines changed

2 files changed

+9
-6
lines changed

configure.ac

+6-6
Original file line numberDiff line numberDiff line change
@@ -155,20 +155,20 @@ AC_ARG_ENABLE(examples,
155155
[SECP_SET_DEFAULT([enable_examples], [no], [yes])])
156156

157157
AC_ARG_ENABLE(module_ecdh,
158-
AS_HELP_STRING([--enable-module-ecdh],[enable ECDH module [default=no]]), [],
159-
[SECP_SET_DEFAULT([enable_module_ecdh], [no], [yes])])
158+
AS_HELP_STRING([--enable-module-ecdh],[enable ECDH module [default=yes]]), [],
159+
[SECP_SET_DEFAULT([enable_module_ecdh], [yes], [yes])])
160160

161161
AC_ARG_ENABLE(module_recovery,
162162
AS_HELP_STRING([--enable-module-recovery],[enable ECDSA pubkey recovery module [default=no]]), [],
163163
[SECP_SET_DEFAULT([enable_module_recovery], [no], [yes])])
164164

165165
AC_ARG_ENABLE(module_extrakeys,
166-
AS_HELP_STRING([--enable-module-extrakeys],[enable extrakeys module [default=no]]), [],
167-
[SECP_SET_DEFAULT([enable_module_extrakeys], [no], [yes])])
166+
AS_HELP_STRING([--enable-module-extrakeys],[enable extrakeys module [default=yes]]), [],
167+
[SECP_SET_DEFAULT([enable_module_extrakeys], [yes], [yes])])
168168

169169
AC_ARG_ENABLE(module_schnorrsig,
170-
AS_HELP_STRING([--enable-module-schnorrsig],[enable schnorrsig module [default=no]]), [],
171-
[SECP_SET_DEFAULT([enable_module_schnorrsig], [no], [yes])])
170+
AS_HELP_STRING([--enable-module-schnorrsig],[enable schnorrsig module [default=yes]]), [],
171+
[SECP_SET_DEFAULT([enable_module_schnorrsig], [yes], [yes])])
172172

173173
AC_ARG_ENABLE(external_default_callbacks,
174174
AS_HELP_STRING([--enable-external-default-callbacks],[enable external default callback functions [default=no]]), [],

doc/CHANGELOG.md

+3
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@ Each change falls into one of the following categories: Added, Changed, Deprecat
66

77
## [Unreleased]
88

9+
### Changed
10+
- Enable modules schnorrsig, extrakeys and ECDH by default in ./configure
11+
912
## [MAJOR.MINOR.PATCH] - YYYY-MM-DD
1013

1114
### Added/Changed/Deprecated/Removed/Fixed/Security

0 commit comments

Comments
 (0)