summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKyren223 <Kyren223@proton.me>2025-01-07 19:05:58 +0200
committerKyren223 <Kyren223@proton.me>2025-01-07 19:05:58 +0200
commitdbea21d9c3642fce6b25837c2ec5fd704a7e8f2a (patch)
tree819aa9a59196132d8157d240e7c1407bbf6b0ce9
parent4e2ca64319f17867ac63a8d1283a67e2f1c4daba (diff)
Fixed all gosec issues
-rw-r--r--.github/workflows/ci.yml2
-rw-r--r--internal/client/gateway/gateway.go4
-rw-r--r--internal/client/ui/auth/auth.go2
-rw-r--r--internal/server/server.go12
-rw-r--r--pkg/assert/assert.go2
5 files changed, 11 insertions, 11 deletions
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index d77b260..66462ed 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -2,7 +2,7 @@ name: ci
on:
push:
- branches: [master, ci]
+ branches: [master]
pull_request:
branches: [master]
diff --git a/internal/client/gateway/gateway.go b/internal/client/gateway/gateway.go
index df1503f..11d15bb 100644
--- a/internal/client/gateway/gateway.go
+++ b/internal/client/gateway/gateway.go
@@ -189,7 +189,7 @@ func handlePacketStream() {
func Disconnect() {
if conn != nil {
- conn.Close()
+ _ = conn.Close()
closed = true
}
}
@@ -200,7 +200,7 @@ func onDisconnect(err error) {
if conn == nil {
return
}
- conn.Close()
+ _ = conn.Close()
conn = nil
close(framer.Out)
if closed {
diff --git a/internal/client/ui/auth/auth.go b/internal/client/ui/auth/auth.go
index 5e44962..4b69735 100644
--- a/internal/client/ui/auth/auth.go
+++ b/internal/client/ui/auth/auth.go
@@ -540,7 +540,7 @@ func (m *Model) signin() tea.Cmd {
}
if m.remember {
- config.Use(func(config *config.Config) {
+ _ = config.Use(func(config *config.Config) {
config.PrivateKeyPath = privateKeyFilepath
})
}
diff --git a/internal/server/server.go b/internal/server/server.go
index 6c915f7..2e5cd7b 100644
--- a/internal/server/server.go
+++ b/internal/server/server.go
@@ -114,7 +114,7 @@ func (s *server) Run() error {
defer listener.Close()
go func() {
<-s.ctx.Done()
- listener.Close()
+ _ = listener.Close()
}()
log.Println("started listening on port", s.Port)
@@ -159,7 +159,7 @@ func (server *server) handleConnection(conn net.Conn) {
if err != nil {
initialCancel()
log.Println(addr, err)
- conn.Close()
+ _ = conn.Close()
log.Println(addr, "disconnected")
return
}
@@ -168,7 +168,7 @@ func (server *server) handleConnection(conn net.Conn) {
if err != nil {
initialCancel()
log.Println(addr, "user creation/fetching error:", err)
- conn.Close()
+ _ = conn.Close()
log.Println(addr, "disconnected")
return
}
@@ -185,7 +185,7 @@ func (server *server) handleConnection(conn net.Conn) {
if err != nil {
initialCancel()
log.Println(addr, "failed to write user id")
- conn.Close()
+ _ = conn.Close()
log.Println(addr, "disconnected")
return
}
@@ -194,10 +194,10 @@ func (server *server) handleConnection(conn net.Conn) {
go func() {
<-ctx.Done()
- conn.Close()
+ _ = conn.Close()
}()
defer func() {
- conn.Close()
+ _ = conn.Close()
sameAddress := addr.String() == server.Session(sess.ID()).Addr().String()
if sameAddress {
server.RemoveSession(sess.ID())
diff --git a/pkg/assert/assert.go b/pkg/assert/assert.go
index 65fe088..083c062 100644
--- a/pkg/assert/assert.go
+++ b/pkg/assert/assert.go
@@ -45,7 +45,7 @@ func runAssert(message string, args ...any) {
flushMu.Lock()
for len(flushes) != 0 {
flusher := flushes[len(flushes)-1]
- flusher.Close()
+ _ = flusher.Close()
flushes = flushes[:len(flushes)-1]
}
flushMu.Unlock()