jwetzell/showbridge-go
1
0
Fork 0
mirror of https://github.com/jwetzell/showbridge-go.git synced 2026-09-09 07:08:58 +00:00
Code Issues Packages Projects Releases 10 Wiki Activity

Merge pull request #153 from jwetzell/fix/serial-client-closing

Browse Source 
close serial client more reliably
This commit is contained in:
Joel Wetzell
2026-05-17 09:06:36 -05:00
committed by GitHub
parent 91063f50da e5c8e01bf0
commit 8752b35ea4
1 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/module/serial-client.go
+6
View File
@@ -125,6 +125,9 @@ func (sc *SerialClient) Start(ctx context.Context, router common.RouteIO) error
if err != nil { if err != nil {
if sc.ctx.Err() != nil { if sc.ctx.Err() != nil {
sc.logger.Debug("done") sc.logger.Debug("done")
if sc.port != nil {
sc.port.Close()
}
return nil return nil
} }
sc.logger.Error("port setup error", "port", sc.Port, "error", err.Error()) sc.logger.Error("port setup error", "port", sc.Port, "error", err.Error())
@@ -136,6 +139,9 @@ func (sc *SerialClient) Start(ctx context.Context, router common.RouteIO) error
select { select {
case <-sc.ctx.Done(): case <-sc.ctx.Done():
sc.logger.Debug("done") sc.logger.Debug("done")
if sc.port != nil {
sc.port.Close()
}
return nil return nil
default: default:
READ: READ: