theforeman / smart_proxy_realm_ad_plugin

foreman-proxy realm plugin for Active Directory
GNU General Public License v3.0
9 stars 10 forks source link

Disabling all modules in the group ['realm_ad', 'realm'] due to a failure in one of them: Parameter 'computername_ prefix' is expected to have a non-empty value #15

Closed RedChops closed 2 months ago

RedChops commented 6 years ago

If I leave computername_prefix unset or commented out in realm_ad.yml, I get the error mentioned in the subject. No clue why, as that setting is not mentioned in the validators. What I ended up doing was modifying configuration_loader.rb with computername_prefix: '' then putting any value for that setting in realm_ad.yml and it seemed to work. This is obviously not a general case fix, but I personally have no use for the field.

I should mention that I'm currently using Foreman 1.18

timogoebel commented 6 years ago

@RedChops: Why didn't you set the computername_prefix in the settings to '', i.e. an empty string?

RedChops commented 6 years ago

@timogoebel That was the first thing I tried, after leaving computername_prefix commented out. Even with it set to '' I get the error mentioned in the subject. I had taken a brief look at the source to see if I could come up with a more general fix, but it didn't look like that field should be getting validated at all so I'm not sure what's up.

martencassel commented 2 months ago

computername_prefix is currently required. The solution is to comment this out. We will probably fix this later.

martencassel commented 2 months ago

can we close this issue ?

RedChops commented 2 months ago

I forgot this ticket was still opened, we haven't used this plugin in nearly 4 years.