Compare commits

...

6 Commits

2 changed files with 156 additions and 66 deletions

View File

@@ -70,6 +70,7 @@ type SafeWebsocketClientBuilder struct {
path *string path *string
rawQuery *string rawQuery *string
useTLS *bool useTLS *bool
channelSize *int64
} }
func NewSafeWebsocketClientBuilder() *SafeWebsocketClientBuilder { func NewSafeWebsocketClientBuilder() *SafeWebsocketClientBuilder {
@@ -101,7 +102,12 @@ func (b *SafeWebsocketClientBuilder) RawQuery(rawQuery string) *SafeWebsocketCli
return b return b
} }
func (b *SafeWebsocketClientBuilder) Build() (*SafeWebsocketClient, error) { func (b *SafeWebsocketClientBuilder) ChannelSize(channelSize int64) *SafeWebsocketClientBuilder {
b.channelSize = &channelSize
return b
}
func (b *SafeWebsocketClientBuilder) Build(ctx context.Context) (*SafeWebsocketClient, error) {
if err := internal.NilChecker(b); err != nil { if err := internal.NilChecker(b); err != nil {
return nil, err return nil, err
} }
@@ -111,19 +117,27 @@ func (b *SafeWebsocketClientBuilder) Build() (*SafeWebsocketClient, error) {
useTLS = *b.useTLS useTLS = *b.useTLS
} }
if b.channelSize == nil {
channelSize := int64(1)
b.channelSize = &channelSize
}
wsClient := SafeWebsocketClient{ wsClient := SafeWebsocketClient{
baseHost: *b.baseHost, baseHost: *b.baseHost,
basePort: *b.basePort, basePort: *b.basePort,
useTLS: useTLS, useTLS: useTLS,
path: b.path, path: b.path,
rawQuery: b.rawQuery, rawQuery: b.rawQuery,
dataChannel: make(chan []byte, 1), dataChannel: make(chan []byte, *b.channelSize),
mu: custom_rwmutex.NewCustomRwMutex(), mu: custom_rwmutex.NewCustomRwMutex(),
ctx: ctx,
reconnectCh: make(chan struct{}, 1), reconnectCh: make(chan struct{}, 1),
isConnected: false, isConnected: false,
doneMap: NewSafeMap[string, chan struct{}](), doneMap: NewSafeMap[string, chan struct{}](),
} }
go wsClient.reconnectHandler()
if err := wsClient.connect(); err != nil { if err := wsClient.connect(); err != nil {
return nil, fmt.Errorf("failed to establish initial connection: %v", err) return nil, fmt.Errorf("failed to establish initial connection: %v", err)
} }
@@ -140,8 +154,8 @@ type SafeWebsocketClient struct {
dataChannel chan []byte dataChannel chan []byte
mu *custom_rwmutex.CustomRwMutex mu *custom_rwmutex.CustomRwMutex
conn *websocket.Conn conn *websocket.Conn
cancelFuncs []context.CancelFunc
ctx context.Context ctx context.Context
cancel context.CancelFunc
reconnectCh chan struct{} reconnectCh chan struct{}
isConnected bool isConnected bool
@@ -174,7 +188,6 @@ func (wsClient *SafeWebsocketClient) connect() error {
} }
conn.SetPingHandler(func(pingData string) error { conn.SetPingHandler(func(pingData string) error {
return wsClient.mu.WriteHandler(func() error {
if err := conn.WriteMessage(websocket.PongMessage, []byte(pingData)); err != nil { if err := conn.WriteMessage(websocket.PongMessage, []byte(pingData)); err != nil {
if err == websocket.ErrCloseSent { if err == websocket.ErrCloseSent {
return nil return nil
@@ -186,73 +199,84 @@ func (wsClient *SafeWebsocketClient) connect() error {
} }
return nil return nil
}) })
})
wsClient.mu.WriteHandler(func() error { wsClient.mu.WriteHandler(func() error {
if wsClient.conn != nil {
wsClient.conn.Close() wsClient.conn.Close()
ctx, cancel := context.WithCancel(context.Background()) }
wsClient.ctx = ctx
wsClient.cancel = cancel
wsClient.conn = conn wsClient.conn = conn
wsClient.isConnected = true wsClient.isConnected = true
pingCtx, pingCancel := context.WithCancel(context.Background())
go wsClient.startPingTicker(pingCtx)
wsClient.cancelFuncs = append(wsClient.cancelFuncs, pingCancel)
receiverCtx, receiverCancel := context.WithCancel(context.Background())
go wsClient.startReceiveHandler(receiverCtx)
wsClient.cancelFuncs = append(wsClient.cancelFuncs, receiverCancel)
return nil return nil
}) })
go wsClient.startPingTicker()
go wsClient.startReceiveHandler()
go wsClient.reconnectHandler()
return nil return nil
} }
func (wsClient *SafeWebsocketClient) startPingTicker() { func (wsClient *SafeWebsocketClient) startPingTicker(ctx context.Context) {
ticker := time.NewTicker(pingPeriod) ticker := time.NewTicker(pingPeriod)
defer ticker.Stop() defer ticker.Stop()
doneKey := "startPingTicker"
wsClient.doneMap.Store(doneKey, make(chan struct{}))
done, _ := wsClient.doneMap.Load(doneKey)
for { for {
select { select {
case <-ctx.Done():
log.Println("ping ticker stopped")
return
case <-ticker.C: case <-ticker.C:
wsClient.mu.WriteHandler(func() error { wsClient.mu.WriteHandler(func() error {
if wsClient.conn == nil {
return fmt.Errorf("connection closed")
}
if err := wsClient.conn.WriteMessage(websocket.PingMessage, []byte{}); err != nil { if err := wsClient.conn.WriteMessage(websocket.PingMessage, []byte{}); err != nil {
log.Printf("Ping failed: %v. Will attempt reconnect.", err) log.Printf("Ping failed: %v. Will attempt reconnect.", err)
wsClient.triggerReconnect() wsClient.triggerReconnect()
} }
return nil return nil
}) })
case <-done:
log.Println("Ping ticker stopped")
return
} }
} }
} }
func (wsClient *SafeWebsocketClient) startReceiveHandler() { func (wsClient *SafeWebsocketClient) startReceiveHandler(ctx context.Context) {
doneKey := "startReceiveHandler"
wsClient.doneMap.Store(doneKey, make(chan struct{}))
done, _ := wsClient.doneMap.Load(doneKey)
for { for {
select { select {
case <-done: case <-ctx.Done():
log.Println("Reconnect handler stopped") log.Println("receive handler stopped")
return return
default: default:
if err := wsClient.mu.ReadHandler(func() error {
conn := wsClient.conn conn := wsClient.conn
if conn == nil { if conn == nil {
wsClient.triggerReconnect() wsClient.triggerReconnect()
return return fmt.Errorf("connection closed")
} }
_, message, err := conn.ReadMessage() _, message, err := conn.ReadMessage()
if err != nil { if err != nil {
wsClient.triggerReconnect() wsClient.triggerReconnect()
return fmt.Errorf("failed to read message: %v", err)
}
select {
case wsClient.dataChannel <- message:
case <-ctx.Done():
log.Println("Reconnect handler stopped")
return nil
default:
log.Println("Data channel full, dropping message")
}
return nil
}); err != nil {
return return
} }
wsClient.dataChannel <- message
} }
} }
} }
@@ -265,20 +289,45 @@ func (wsClient *SafeWebsocketClient) triggerReconnect() {
} }
func (wsClient *SafeWebsocketClient) reconnectHandler() { func (wsClient *SafeWebsocketClient) reconnectHandler() {
doneKey := "reconnectHandler" backoff := 1 * time.Second
wsClient.doneMap.Store(doneKey, make(chan struct{})) maxBackoff := 30 * time.Second
done, _ := wsClient.doneMap.Load(doneKey)
for { for {
select { select {
case <-wsClient.reconnectCh: case <-wsClient.reconnectCh:
wsClient.cancel() log.Println("Reconnect triggered")
wsClient.connect() if wsClient.cancelFuncs != nil {
for _, cancel := range wsClient.cancelFuncs {
cancel()
}
}
wsClient.isConnected = false
wsClient.doneMap.Range(func(s string, c chan struct{}) bool { time.Sleep(100 * time.Millisecond)
c <- struct{}{}
return true isInnerLoop := true
}) for isInnerLoop {
case <-done: log.Printf("Attempting reconnect in %v...", backoff)
select {
case <-time.After(backoff):
if err := wsClient.connect(); err != nil {
log.Printf("Reconnect failed: %v", err)
if backoff < maxBackoff {
backoff *= 2
}
continue
}
log.Println("Reconnected successfully")
backoff = 1 * time.Second
isInnerLoop = false
continue
case <-wsClient.ctx.Done():
return
}
}
case <-wsClient.ctx.Done():
log.Println("reconnect handler stopped due to client shutdown")
wsClient.Close()
return return
} }
} }
@@ -287,3 +336,27 @@ func (wsClient *SafeWebsocketClient) reconnectHandler() {
func (wsClient *SafeWebsocketClient) DataChannel() <-chan []byte { func (wsClient *SafeWebsocketClient) DataChannel() <-chan []byte {
return wsClient.dataChannel return wsClient.dataChannel
} }
func (wsClient *SafeWebsocketClient) WriteJSON(message any) error {
return wsClient.mu.WriteHandler(func() error {
return wsClient.conn.WriteJSON(message)
})
}
func (wsClient *SafeWebsocketClient) Close() error {
wsClient.mu.WriteHandler(func() error {
if wsClient.cancelFuncs != nil {
for _, cancel := range wsClient.cancelFuncs {
cancel()
}
}
if wsClient.conn != nil {
wsClient.conn.Close()
}
wsClient.isConnected = false
return nil
})
close(wsClient.dataChannel)
return nil
}

View File

@@ -1,22 +1,39 @@
package main package main
import ( import (
"context"
"fmt" "fmt"
"log" "log"
"os"
"os/signal"
"syscall"
"git.neurocipta.com/rogerferdinan/safe-web-socket/v1/client" "git.neurocipta.com/rogerferdinan/safe-web-socket/v1/client"
) )
func main() { func main() {
sigChan := make(chan os.Signal, 1)
signal.Notify(sigChan, syscall.SIGINT, syscall.SIGTERM)
ctx, cancel := context.WithCancel(context.Background())
defer cancel()
go func() {
<-sigChan
fmt.Println("\nReceived interrupt signal. Shutting down gracefully...")
cancel()
}()
wsClient, err := client.NewSafeWebsocketClientBuilder(). wsClient, err := client.NewSafeWebsocketClientBuilder().
BaseHost("localhost"). BaseHost("localhost").
BasePort(8080). BasePort(8080).
Path("/ws/test/data_1"). Path("/ws/test/data_1").
UseTLS(false). UseTLS(false).
Build() Build(ctx)
if err != nil { if err != nil {
log.Fatal(err) log.Fatal(err)
} }
dataChannel := wsClient.DataChannel() dataChannel := wsClient.DataChannel()
for data := range dataChannel { for data := range dataChannel {