-
Notifications
You must be signed in to change notification settings - Fork 155
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
Sanitize config option in AdaptPipeline initialization #159
Comments
@JarbasAl did you intend to open this against the adapt repo? It seems more relevant to the pipeline code. |
Sorry, the bot got confused with the fork apparently, it asked of I wanted it to open an issue and I said yes, didn't realize it was opened here |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The method in the
AdaptPipeline
class needs to sanitize the config option as indicated by the TODO comment. This is to ensure that the configuration settings are handled securely and correctly.PR URL: OpenVoiceOS#2
Comment URL: OpenVoiceOS#2 (comment)
Please address this issue to improve the robustness of the initialization logic.
The text was updated successfully, but these errors were encountered: