Skip to content

Conversation

f0ssel
Copy link
Collaborator

@f0ssel f0ssel commented Sep 10, 2025

Problem

The --no-tls-intercept flag was present in the CLI but not fully utilized or documented, creating confusion about TLS handling capabilities.

Solution

Removed the unused TLS flag to simplify the CLI interface and reduce confusion.

Changes

  • Removed --no-tls-intercept flag from CLI options
  • Simplified TLS certificate manager initialization
  • Cleaned up unused TLS configuration paths

Benefits

  • 🧹 Cleaner CLI: Removes confusing unused flag
  • 📚 Better UX: Eliminates unclear options for users
  • 🔧 Simplified code: Reduces complexity in CLI handling

Testing

  • ✅ Build succeeds
  • ✅ All tests pass
  • ✅ CLI functionality unchanged
  • ✅ TLS interception continues to work as expected

@blink-so-v1 blink-so-v1 bot changed the title remove tls flag fix: remove unused TLS interception flag Sep 10, 2025
@f0ssel f0ssel merged commit eaf3370 into main Sep 10, 2025
7 checks passed
@f0ssel f0ssel deleted the f0ssel/no-tls-flag branch September 10, 2025 20:40
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