Setting `host` in `Configuration` causes credentials to be sent in the clear
まだ誰も着手していません。
評価
調査の方向性
レポートでは Configuration(host=...)、ApiClient(configuration)、MonitorsApi.list_monitors() が挙げられていますが、ソースファイルやテストパスは示されていません。まず、設定されたホストがリクエストスキームをどのように選択するかを追跡し、デフォルトホストのパスと比較してください。明示的に設定されたホストがデフォルトで TLS を使用し、認証情報が初回の HTTP リクエストで決して送信されないことを明示的に確認できれば完了です。再現したケースの回帰テストを追加してください。
索引モデルが issue の本文から書いたものです。
説明
Describe the bug
Setting host in the Configuration ctor causes credentials to be sent in the clear.
To Reproduce
Take this example:
import os
from datadog_api_client import ApiClient, Configuration
from datadog_api_client.v1.api.monitors_api import MonitorsApi
configuration = Configuration(
host=os.environ['DATADOG_HOST'],
)
configuration.api_key["apiKeyAuth"] = os.environ['DATADOG_API_KEY']
configuration.api_key["appKeyAuth"] = os.environ['DATADOG_APP_KEY']
configuration.debug = True
with ApiClient(configuration) as api_client:
monitors_api = MonitorsApi(api_client)
monitors = monitors_api.list_monitors()
print('{} monitors'.format(len(monitors)))
You can tell from the debug output that we're not doing TLS:
send: b'GET /api/v1/monitor HTTP/1.1\r\nHost: <subdomain>.datadoghq.com\r\nAccept: application/json\r\n<snip>\r\n\r\n'
reply: 'HTTP/1.1 308 Permanent Redirect\r\n'
header: Date: Fri, 08 Dec 2023 23:22:03 GMT
header: Content-Length: 0
header: Connection: keep-alive
header: location: https://<subdomain>.datadoghq.com/api/v1/monitor
header: x-content-type-options: nosniff
header: strict-transport-security: max-age=31536000; includeSubDomains; preload
send: b'GET /api/v1/monitor HTTP/1.1\r\nHost: <subdomain>.datadoghq.com\r\nAccept: application/json\r\n<snip>\r\n\r\n'
reply: 'HTTP/1.1 200 OK\r\n'
header: Date: Fri, 08 Dec 2023 23:22:03 GMT
header: Content-Type: application/json
header: Transfer-Encoding: chunked
header: Connection: keep-alive
header: etag: W/"721e351f3b20293a0d2bc1301301df83"
header: x-frame-options: SAMEORIGIN
header: content-security-policy: frame-ancestors 'self'; report-uri https://logs.browser-intake-datadoghq.com/api/v2/logs?dd-api-key=pube4f163c23bbf91c16b8f57f56af9fc58&dd-evp-origin=content-security-policy&ddsource=csp-report&ddtags=site%3Adatadoghq.com
header: vary: Accept-Encoding
header: content-encoding: gzip
header: x-ratelimit-limit: 1000
header: x-ratelimit-period: 10
header: x-ratelimit-remaining: 999
header: x-ratelimit-reset: 7
header: x-ratelimit-name: get_all_monitors
header: x-content-type-options: nosniff
header: strict-transport-security: max-age=31536000; includeSubDomains; preload
240 monitors
That double request is pretty clearly an HTTP→HTTPS redirect. Wireshark confirms as much. Oddly, whatever the default for host is doesn't do this, but I didn't realize until much later that setting host isn't required, see the Additional Context section. Initially, I was under the impression that configuration of what DD tenant I was was required.
Expected behavior
TLS is enabled by default … and ideally only switch off very explicitly. Certainly overriding things like hostname don't then also remove TLS, in addition…
Environment and Versions (please complete the following information):
A clear and precise description of your setup:
- version for this project in use: 2.19.0
- services, libraries, languages and tools list and versions: the minimal example above
Additional context
I ended up setting host almost by accident. Initially, I was getting 403 Forbidden from the API, and wasn't sure why; I was using the "Getting Started" example, and setting an API key. It turns out DD requires both an API key and an "app" key (the README does note this, but I missed that in my first pass), which is somewhat unusual. So, AFAICT, I wasn't authenticated, but since the wrong HTTP status was getting returned, that never clicked, and I tried other things (such as configuring my host) prior to returning to the what-if of "perhaps it is, indeed, authn?"
I'm not clear on why one can construct a DD client without (full) credentials without error in the first place.
- 主要言語
- Python
- スター
- 166
- フォーク
- 55
- 平均マージ
- 2日 17時間
- マージ済み PR(30日)
- 73
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
DataDog/datadog-api-client-python のほかの issue
-
stale
難易度 1/5 1〜3時間 初心者へのやさしさ 72/100
DataDog/datadog-api-client-python#3535 · コメント 1 件 ·
-
kind/bug stale
難易度 3/5 1〜2日 初心者へのやさしさ 66/100
DataDog/datadog-api-client-python#3717 · コメント 3 件 ·
-
kind/bug stale
難易度 3/5 1〜2日 初心者へのやさしさ 75/100
DataDog/datadog-api-client-python#3656 · コメント 1 件 ·
-
kind/bug stale
難易度 3/5 1〜2日 初心者へのやさしさ 38/100
DataDog/datadog-api-client-python#3120 · コメント 1 件 ·
-
stale
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
DataDog/datadog-api-client-python#2986 · コメント 1 件 ·
DataDog/datadog-api-client-python の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
anthropics/skills#1811 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
speaches-ai/speaches#678 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
datalayer/mcp-compose#42 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
conda-forge/spacy-feedstock#177 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
UKGovernmentBEIS/inspect_evals#2523 ·