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.

Sign up for GitHub

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

Jump to bottom

Refactor configuration parsing #13

Open
hariso opened this issue Mar 25, 2022 · 0 comments
Open

Refactor configuration parsing #13

hariso opened this issue Mar 25, 2022 · 0 comments
Labels
feature New feature or request

Comments

@hariso
Copy link
Contributor

hariso commented Mar 25, 2022 •
edited
Loading

The connector's configuration consists of a number of parameters of different types and with different validations. Currently, we're parsing fields one by one. If the configuration continues growing, we should look into a more generic way to parse configurations, to keep maintainable code.

Another thing to consider here are parameters for explicitly enabing/disabling TLS. Currently, TLS is considered enabled if any TLS setting is present.
@hariso hariso added feature New feature or request triage labels Mar 25, 2022
@hariso hariso mentioned this issue Mar 25, 2022
Merged
4 tasks
@lovromazgon lovromazgon moved this to Triage in Conduit Main Mar 30, 2022
@lovromazgon lovromazgon added this to the Future milestone Apr 11, 2022
@hariso hariso mentioned this issue Apr 14, 2022
Merged
4 tasks
@raulb raulb removed this from the Future milestone Feb 8, 2025
@github-project-automation github-project-automation bot moved this to Triage in Conduit Main Feb 8, 2025
@raulb raulb removed the status in Conduit Main Feb 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature New feature or request
Projects
Conduit Main
Status: No status
Milestone
No milestone
Development

No branches or pull requests
3 participants
@raulb @hariso @lovromazgon