influxdata / openapi

An OpenAPI specification for influx (cloud/oss) apis.
MIT License
17 stars 10 forks source link

fix(user): POST operation for `/users/{userID}/password` doesn't require basic auth #533

Closed bednar closed 1 year ago

bednar commented 1 year ago

Related to https://influxcommunity.slack.com/archives/CHQ5VG6F8/p1662739448171649

The POST operation for /users/{userID}/password is used by InfluxDB clients (for more info see https://github.com/influxdata/influxdb-client-python/pull/498) and with current oss.yml we can't update the user's password.

This PR change the contracts according to current OSS implementation:

For more info see:

jstirnaman commented 1 year ago

Adding @jeffreyssmith2nd and @lesam just to be sure, but it LGTM - we can improve the docs.

bednar commented 1 year ago

The PR is rebased

lesam commented 1 year ago

This looks like one for @jeffreyssmith2nd

bednar commented 1 year ago

@jstirnaman thanks for review. The description and examples are updated.

bednar commented 1 year ago

@jstirnaman thanks for your approval. I don't have a permission to merge this PR into master... Can you merge this PR? Thanks a lot