Skip to content

fix(serve): keep proxy credentials out of the ignored-params warning - #125

Merged
tenequm merged 2 commits into
mainfrom
fix/mask-proxy-in-log
Sep 30, 2026
Merged

tenequm merged 2 commits into
mainfrom
fix/mask-proxy-in-log

Conversation

@tenequm

@tenequm tenequm commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

When a connection asks for a seed that is already running with different params, the daemon warns that the new params are ignored and names the running seed's proxy. It printed the proxy URL as given, so a proxy with credentials put its username and password in the log, and the session log is teed to the profile volume.

The warning now names the proxy without its userinfo (http://host:port), for http, https and SOCKS proxies alike. This was the only log line that printed a proxy.

Release notes

The "seed already running - ignoring new params" warning no longer prints the proxy's username and password; it shows only the proxy's scheme, host and port.

@tenequm
tenequm merged commit 2cf0023 into main Sep 30, 2026
9 checks passed
@tenequm
tenequm deleted the fix/mask-proxy-in-log branch September 30, 2026 00:48
@tenequm tenequm self-assigned this Sep 30, 2026
tenequm added a commit that referenced this pull request Sep 30, 2026
🤖 I have created a release *beep* *boop*
---


## [0.16.0](v0.15.1...v0.16.0)
(2026-09-30)


### ⚠ BREAKING CHANGES

* **browser:** stealth Chromium 154 with coherent personas
([#124](#124))

### Features

* **browser:** stealth Chromium 154 with coherent personas
([#124](#124))
([6eacdd7](6eacdd7))


### Bug Fixes

* **serve:** keep proxy credentials out of the ignored-params warning
([#125](#125))
([2cf0023](2cf0023))
* **skill:** ship the skill as a real file under skills/cuttle
([#122](#122))
([e04bd29](e04bd29))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant