forked from namjaejeon/ksmbd-tools
-
Notifications
You must be signed in to change notification settings - Fork 27
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
Sanity-check configuration file #248
Comments
Agreed, Good suggestion:), samba also ignores unsupported parameters without a message. but ksmbd is a latecomer. As you said, it seems to be needed. Let me check it. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi, everyone.
I want to suggest a feature that's equivalent to Samba's testparm to sanity-check the configuration file.
While the documentation on Documentation/configuration.txt is quite comprehensive, it seems like ksmbd does not alert the users of any unrecognized values in the config file.
It would be a nice addition to ksmbd-tools (or ksmbd itself - e.g., warning in the kernel log) for new comers, migrating from other setups.
Thanks.
The text was updated successfully, but these errors were encountered: