Skip to content

Commit 8480631

Browse files
committed
refactor(firewall): improve port completion list
- Remove FTP (21) - not recommended for production use - Remove all Dev-Server entries (3000, 5000) - not relevant for firewall rules - Update descriptions for ports 80, 8000, 8080 to clarify HTTP/HTTPS usage Changed from 'HTTP-Alt' to 'HTTP (or HTTPS w/TLS)' for clarity
1 parent 8c1fcdf commit 8480631

1 file changed

Lines changed: 3 additions & 6 deletions

File tree

internal/commands/server/firewall/completion.go

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -37,21 +37,18 @@ func parseServicesFile() []string {
3737
desc string
3838
}{
3939
{"22", "SSH"},
40-
{"80", "HTTP"},
40+
{"80", "HTTP (or HTTPS w/TLS)"},
4141
{"443", "HTTPS"},
42-
{"21", "FTP"},
4342
{"25", "SMTP"},
4443
{"53", "DNS"},
4544
{"110", "POP3"},
4645
{"143", "IMAP"},
4746
{"3306", "MySQL"},
4847
{"5432", "PostgreSQL"},
4948
{"6379", "Redis"},
50-
{"8080", "HTTP-Alt"},
49+
{"8080", "HTTP (or HTTPS w/TLS)"},
5150
{"8443", "HTTPS-Alt"},
52-
{"3000", "Dev-Server"},
53-
{"5000", "Dev-Server"},
54-
{"8000", "Dev-Server"},
51+
{"8000", "HTTP (or HTTPS w/TLS)"},
5552
}
5653

5754
for _, cp := range commonPorts {

0 commit comments

Comments
 (0)