diff mbox series

[1/1] freeradius-server: enable default modules mschap and realm

Message ID 20231031225751.35130-1-brandon.maier@collins.com
State Accepted
Headers show
Series [1/1] freeradius-server: enable default modules mschap and realm | expand

Commit Message

Brandon Maier Oct. 31, 2023, 10:57 p.m. UTC
From: Kalpesh Panchal <kalpesh.panchal2@collins.com>

"mschap" and "realm" are already part of default required
raddb modules(DEFAULT_MODULES in raddb/all.mk).

so removed them from disabled module list so that
radiusd.service can start without following errors:

- /etc/raddb/mods-enabled/mschap[11]: Failed to link to module 'rlm_mschap': /usr/lib/rlm_mschap.so
- /etc/raddb/mods-enabled/realm[21]: Failed to link to module 'rlm_realm': /usr/lib/rlm_realm.so

Signed-off-by: Kalpesh Panchal <kalpesh.panchal2@collins.com>
Signed-off-by: Brandon Maier <brandon.maier@collins.com>
---
 package/freeradius-server/freeradius-server.mk | 2 --
 1 file changed, 2 deletions(-)

Comments

Thomas Petazzoni Nov. 4, 2023, 6:24 p.m. UTC | #1
On Tue, 31 Oct 2023 22:57:51 +0000
Brandon Maier via buildroot <buildroot@buildroot.org> wrote:

> From: Kalpesh Panchal <kalpesh.panchal2@collins.com>
> 
> "mschap" and "realm" are already part of default required
> raddb modules(DEFAULT_MODULES in raddb/all.mk).
> 
> so removed them from disabled module list so that
> radiusd.service can start without following errors:
> 
> - /etc/raddb/mods-enabled/mschap[11]: Failed to link to module 'rlm_mschap': /usr/lib/rlm_mschap.so
> - /etc/raddb/mods-enabled/realm[21]: Failed to link to module 'rlm_realm': /usr/lib/rlm_realm.so
> 
> Signed-off-by: Kalpesh Panchal <kalpesh.panchal2@collins.com>
> Signed-off-by: Brandon Maier <brandon.maier@collins.com>
> ---
>  package/freeradius-server/freeradius-server.mk | 2 --
>  1 file changed, 2 deletions(-)

Applied to master, thanks. It would be good to have a test case in
support/testing/ for freeradius, to make sure it starts up correctly at
boot.

Thomas
Peter Korsgaard Nov. 9, 2023, 11:12 a.m. UTC | #2
>>>>> "Brandon" == Brandon Maier via buildroot <buildroot@buildroot.org> writes:

 > From: Kalpesh Panchal <kalpesh.panchal2@collins.com>
 > "mschap" and "realm" are already part of default required
 > raddb modules(DEFAULT_MODULES in raddb/all.mk).

 > so removed them from disabled module list so that
 > radiusd.service can start without following errors:

 > - /etc/raddb/mods-enabled/mschap[11]: Failed to link to module 'rlm_mschap': /usr/lib/rlm_mschap.so
 > - /etc/raddb/mods-enabled/realm[21]: Failed to link to module 'rlm_realm': /usr/lib/rlm_realm.so

 > Signed-off-by: Kalpesh Panchal <kalpesh.panchal2@collins.com>
 > Signed-off-by: Brandon Maier <brandon.maier@collins.com>

Committed to 2023.02.x and 2023.08.x, thanks.
diff mbox series

Patch

diff --git a/package/freeradius-server/freeradius-server.mk b/package/freeradius-server/freeradius-server.mk
index af3320eb44..125427c9e2 100644
--- a/package/freeradius-server/freeradius-server.mk
+++ b/package/freeradius-server/freeradius-server.mk
@@ -45,9 +45,7 @@  FREERADIUS_SERVER_CONF_OPTS += \
 FREERADIUS_SERVER_CONF_OPTS += \
 	--without-rlm_eap_ike \
 	--without-rlm_eap_tnc \
-	--without-rlm_mschap \
 	--without-rlm_perl \
-	--without-rlm_realm \
 	--without-rlm_sql_iodbc \
 	--without-rlm_sql_oracle \
 	--without-rlm_sql_freetds \