Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BREAKING CHANGE] Replace toggle-privacy key binding with expand-privacy and contract-privacy #1347

Closed
fujiapple852 opened this issue Oct 9, 2024 · 2 comments · Fixed by #1353
Labels
breaking A breaking change enhancement New feature or request tui
Milestone

Comments

@fujiapple852
Copy link
Owner

fujiapple852 commented Oct 9, 2024

Add expand-privacy and contract-privacy key bindings which increase and decrease tui-privacy-max-ttl. The value is clamped in the range 0 (i.e. off) and the current maximum ttl of the trace.

Deprecation

This is a breaking change as the existing toggle-privacy key binding has been deprecated.

If used from a configuration file it will result in an error:

$ sudo ./target/debug/trip 1.1.1.1
Error: toggle-privacy in [bindings] section is deprecated, use expand-privacy and contract-privacy instead

If used as a command line parameter (unlikely) it will result in an error:

$ sudo ./target/debug/trip 1.1.1.1 --tui-key-bindings toggle-privacy=o
error: invalid value 'toggle-privacy=o' for '--tui-key-bindings <TUI_KEY_BINDINGS>': toggle-privacy is deprecated, use expand-privacy and contract-privacy instead

For more information, try '--help'.
@fujiapple852 fujiapple852 added enhancement New feature or request triage labels Oct 9, 2024
@fujiapple852 fujiapple852 added this to the 0.12.0 milestone Oct 9, 2024
@fujiapple852 fujiapple852 self-assigned this Oct 9, 2024
@fujiapple852 fujiapple852 changed the title Add tui-privacy config and change default value of tui-privacy-max-ttl [BREAKING CHANGE] Add tui-privacy config and change default value of tui-privacy-max-ttl Oct 14, 2024
@fujiapple852
Copy link
Owner Author

Description of the feature from the 0.9.0 release note here.

fujiapple852 added a commit that referenced this issue Oct 14, 2024
fujiapple852 added a commit that referenced this issue Oct 14, 2024
fujiapple852 added a commit that referenced this issue Oct 15, 2024
fujiapple852 added a commit that referenced this issue Oct 15, 2024
fujiapple852 added a commit that referenced this issue Oct 15, 2024
fujiapple852 added a commit that referenced this issue Oct 15, 2024
fujiapple852 added a commit that referenced this issue Oct 15, 2024
fujiapple852 added a commit that referenced this issue Oct 15, 2024
fujiapple852 added a commit that referenced this issue Oct 15, 2024
fujiapple852 added a commit that referenced this issue Oct 15, 2024
fujiapple852 added a commit that referenced this issue Oct 15, 2024
@fujiapple852 fujiapple852 changed the title [BREAKING CHANGE] Add tui-privacy config and change default value of tui-privacy-max-ttl [BREAKING CHANGE] Add expand-privacy and contract-privacy bindings Oct 15, 2024
@fujiapple852 fujiapple852 changed the title [BREAKING CHANGE] Add expand-privacy and contract-privacy bindings [BREAKING CHANGE] Replace toggle-privacy key binding with expand-privacy and contract-privacy Oct 15, 2024
fujiapple852 added a commit that referenced this issue Oct 15, 2024
fujiapple852 added a commit that referenced this issue Oct 15, 2024
fujiapple852 added a commit that referenced this issue Oct 15, 2024
fujiapple852 added a commit that referenced this issue Oct 15, 2024
@fujiapple852 fujiapple852 added tui breaking A breaking change labels Oct 15, 2024
fujiapple852 added a commit that referenced this issue Oct 16, 2024
fujiapple852 added a commit that referenced this issue Oct 16, 2024
fujiapple852 added a commit that referenced this issue Oct 16, 2024
fujiapple852 added a commit that referenced this issue Oct 16, 2024
@fujiapple852 fujiapple852 removed their assignment Oct 27, 2024
@fujiapple852
Copy link
Owner Author

Related change: #1365

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking A breaking change enhancement New feature or request tui
Projects
None yet
1 participant