17546336088

Ran 08 Sep 2025 09:34AM UTC

Jobs 0

Files 0

Run time –

Badge

Embed ▾

pending completion

Build # 17546336088

Build Type

push

github

Committed by

web-flow

Commit Message

feat: default to Basic Auth header; stop sending creds in URL/query (#731)

* feat(auth): default to Basic Auth header; stop sending creds in URL/query

Build Authorization: Basic … header in Pool when auth is provided
Pass auth from InfluxDB for all requests (/query, /write, ping)
Remove u/p query params; never append credentials to URLs
Keep DSN parsing (user:pass@host) working; now used only for header
Aligns with InfluxDB v1 auth guidance and avoids leaking creds in logs/proxies
Refs node-influx/node-influx#323, node-influx/node-influx#264
Docs: https://docs.influxdata.com/influxdb/v1/administration/authentication_and_authorization/

* test(auth): update unit tests for Basic Auth via headers

Adjust test/unit/influx.test.ts helpers to expect auth option and no u/p query params
Add /pool/echo-headers route in test/fixture/pool-middleware.js to inspect request headers
Add pool test asserting Authorization: Basic … is sent when auth is provided
Refs: node-influx/node-influx#323, node-influx/node-influx#264

---------

Co-authored-by: Paul Deng <paul.deng@gallagher.com>

Run Details

Source Files on build 17546336088

Detailed source file information is not available for this build.

node-influx / node-influx / 17546336088

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 17546336088

node-influx / node-influx / 17546336088

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 17546336088

README BADGES
x