Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

netbios_name default string #55

Open
nassarmu opened this issue Sep 26, 2024 · 0 comments
Open

netbios_name default string #55

nassarmu opened this issue Sep 26, 2024 · 0 comments

Comments

@nassarmu
Copy link

The default netbios_name value results in a literal %{facts.hostname} in the netbios name field. Probably because the value is using single quotes instead of double quotes. When modifying this you should consider using the modern fact %{facts.networking.hostname} as the bare %{facts.hostname} is a legacy fact.

Variant[Undef, String] $netbios_name = '%{facts.hostname}',

dadez73 pushed a commit to dadez73/puppet-samba that referenced this issue Oct 9, 2024
Add inherit_acl and host msdfs at the share level
Fix netbios_name default string rehanone#55
regenerated REFERENCE.md
dadez73 pushed a commit to dadez73/puppet-samba that referenced this issue Oct 14, 2024
Add hosts_allow and host_deny at the share level
Add inherit_acl and host msdfs at the share level
Fix netbios_name default string rehanone#55
regenerated REFERENCE.md

added +4 in samba test resource
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant