search

MycroftAI / adapt

Adapt Intent Parser
Apache License 2.0
712 stars 154 forks source link

Sanitize config option in AdaptPipeline initialization #159

Open coderabbitai[bot] opened 1 month ago

coderabbitai[bot] commented 1 month ago

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: https://github.com/OpenVoiceOS/ovos-adapt-pipeline-plugin/pull/2 Comment URL: https://github.com/OpenVoiceOS/ovos-adapt-pipeline-plugin/pull/2#discussion_r1685505255

Please address this issue to improve the robustness of the initialization logic.

clusterfudge commented 1 month ago

@JarbasAl did you intend to open this against the adapt repo? It seems more relevant to the pipeline code.

JarbasAl commented 1 month ago

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