Skip to content

Commit 0c90918

Browse files
committed
Don't attempt to close Sender on error
Resolves: rhbz#2179924
1 parent 00d2af2 commit 0c90918

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

connector/amqp10/amqp10.go

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -542,11 +542,11 @@ func (conn *AMQP10Connector) Start(outchan chan interface{}, inchan chan interfa
542542
} else {
543543
if s, err := conn.CreateSender(message.Address); err != nil {
544544
conn.logger.Metadata(logging.Metadata{
545-
"cause": "creating sender failed",
546-
"reason": err,
545+
"cause": "creating sender failed",
546+
"reason": err,
547+
"address": message.Address,
547548
})
548549
conn.logger.Warn("Skipping processing message")
549-
(*s).Close(nil)
550550

551551
lfLock.Lock()
552552
linkFail += 1
@@ -562,8 +562,9 @@ func (conn *AMQP10Connector) Start(outchan chan interface{}, inchan chan interfa
562562
if err := (*sender).Error(); err != nil {
563563
// verify sender connection
564564
conn.logger.Metadata(logging.Metadata{
565-
"cause": "sender disconnected",
566-
"reason": err,
565+
"cause": "sender disconnected",
566+
"reason": err,
567+
"address": message.Address,
567568
})
568569
conn.logger.Warn("Skipping processing message")
569570

0 commit comments

Comments
 (0)