Skip to content

feat: edit forwarded ports individually - #9

Open
dr-hoseyn wants to merge 2 commits into
AminMGMT:mainfrom
dr-hoseyn:fix/edit-forwarded-ports
Open

feat: edit forwarded ports individually#9
dr-hoseyn wants to merge 2 commits into
AminMGMT:mainfrom
dr-hoseyn:fix/edit-forwarded-ports

Conversation

@dr-hoseyn

Copy link
Copy Markdown

Summary

  • add dedicated actions to add, edit, or remove one forwarded-port entry
  • show the current entry as the default while editing, so the rest of the list does not need to be re-entered
  • keep full-list replacement available for bulk changes
  • keep hidden Telegram/SOCKS relay mappings preserved through the existing EditTunnel path
  • validate individual edits before saving and prevent removal of the last forwarded port

Testing

  • gofmt on changed Go files
  • GOOS=linux GOARCH=amd64 go test -c ./internal/manage
  • added unit coverage for add/edit/remove and invalid inputs

Note: current main already contains config hot reload in cmd/reload.go; this PR addresses the remaining CLI usability issue where changing one forwarded port required retyping the entire list.

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