Skip to content

Warn on ignored request flags during inspect modes#775

Merged
ryanfowler merged 1 commit into
mainfrom
warn-on-ignored-inspect-flags
Jun 16, 2026
Merged

Warn on ignored request flags during inspect modes#775
ryanfowler merged 1 commit into
mainfrom
warn-on-ignored-inspect-flags

Conversation

@ryanfowler

Copy link
Copy Markdown
Owner

Summary

  • Update --inspect-dns and --inspect-tls to warn with clearer wording when request-only CLI flags are supplied.
  • Expand the ignored-flag detection to cover additional request, transport, formatting, and WebSocket options.
  • Keep config-file defaults from triggering the warning so only explicit CLI flags are reported.
  • Refresh docs to describe the new warning behavior.

Testing

  • Added integration coverage for both inspection modes, including explicit CLI flags and config-backed defaults.
  • Verified the new warning text and ensured the older --inspect-*- ignores wording no longer appears.
  • Existing inspection behavior remains unchanged when no ignored CLI flags are present.

@ryanfowler ryanfowler enabled auto-merge June 16, 2026 04:36
@ryanfowler ryanfowler merged commit c44b523 into main Jun 16, 2026
4 checks passed
@ryanfowler ryanfowler deleted the warn-on-ignored-inspect-flags branch June 16, 2026 04:39
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