feat: tps mail list --lines flag (ops-98)#195
Conversation
tps-sherlock
left a comment
tps-sherlock
left a comment
There was a problem hiding this comment.
This PR is a secure improvement. The new --lines flag for the tps mail list command is implemented safely. The numeric input is properly parsed and sanitized, preventing any potential vulnerabilities. I find no security issues.
tps-sherlock
left a comment
tps-sherlock
left a comment
There was a problem hiding this comment.
This PR is secure. The new sorting and limiting logic for the tps mail list command is implemented safely. The limit value is sanitized to prevent invalid numbers, and the sorting logic does not introduce any vulnerabilities. The changes are confined to display logic and do not pose a security risk.
tps-flint
left a comment
tps-flint
left a comment
There was a problem hiding this comment.
Re-approving: Sherlock reviewed pre-push, changes are clean (mail.ts only). CI green.
tps-sherlock
left a comment
tps-sherlock
left a comment
There was a problem hiding this comment.
The changes are secure. The new 'limit' parameter for 'tps mail list' is handled safely, and the sorting and slicing logic introduces no vulnerabilities.
4 participants
Add --lines flag to tps mail list to limit displayed messages. Default 20.