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

Transport options API #216

Closed
offa opened this issue Nov 13, 2023 · 1 comment · Fixed by #220
Closed

Transport options API #216

offa opened this issue Nov 13, 2023 · 1 comment · Fixed by #220
Assignees
Milestone

Comments

@offa
Copy link
Owner

offa commented Nov 13, 2023

The configuration string is too limited for certain features (see discussion) and an API for different types of options is needed.

Related:

@offa offa added the feature label Nov 13, 2023
@offa offa added this to the v0.8.0 milestone Nov 13, 2023
@offa
Copy link
Owner Author

offa commented Nov 27, 2023

A possible implementation could look like this (using some of the related issues for demonstration):

auto db = influxdb::http("http://localhost:8086?db=test")
    .setProxy(…)
    .setTimeout(10_s)
    .setVerifyCert(false)
    .connect();

This would enable settings for Http transport only and not alter the Transport interface for all classes. What do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant