-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[exporter/influxdb] Update README.md to explain URL path behavior #12794
Conversation
Updated the documentation of the mandatory endpoint parameter to specified the behavior when no path (or root path) is specified as the example doesn't include a path and the resulting behavior in that scenario (insertion of path /api/v2/write) can be surprising. Additionally, updated endpoint description to note HTTP or HTTPS can be used (as example and documentation only referenced HTTP).
|
@clarkington3, thank you for the contribution. Please sign the CLA |
Done, thanks! |
@jacobmarble, please review as code owner |
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
@jacobmarble, please review as code owner |
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
@djaglowski Is it possible to get someone else to review my documentation change? |
@clarkington3, let's give the code owner 2 more weeks. If this has not been reviewed, we will mark it as an unmaintained component. cc @jacobmarble |
Description: Updated the documentation of the mandatory endpoint parameter to clarify the behavior when no path (or root path) is specified as the example doesn't include a path and the resulting behavior in that scenario (insertion of path /api/v2/write) can be surprising. Additionally, updated endpoint description to note HTTP or HTTPS can be used (as example and documentation only referenced HTTP).
Link to tracking Issue: N/A
Testing: Confirmed behavior continues to work consistently - nothing is broken, merely documenting functionality a little more explicitly.
Documentation: Made two changes to README.md file to add information on endpoint URL path functionality and that endpoint supports both HTTP and HTTPS.