djjudas21 / puppet-freeradius

Puppet module to install and configure FreeRADIUS
Apache License 2.0
8 stars 49 forks source link

Fix locking in freeradius::module::detail #211

Closed amateo closed 11 months ago

amateo commented 11 months ago

The locking parameter is a Enum['no', 'yes'], so it can't be checked neither assigned as a boolean. The assignment must be to the value of the parameter, if present.

amateo commented 11 months ago

PDK test is failing, but I think the error is not related with my commit.

djjudas21 commented 11 months ago

The PDK test failure looks unrelated, so I'm happy to override it and merge this. We'll worry about the tests another day :upside_down_face: Thanks for the fix!