Nail/fission-src
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix Kafka message queue trigger panic problem caused by err.Error() (#1271)

Browse Source 
fix bug: panic: runtime error: invalid memory address or nil pointer dereference
This commit is contained in:
cocoifly10
2019-08-14 17:07:29 +08:00
committed by Ta-Ching Chen
parent a6e75b4da6
commit 88b5343775
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/mqtrigger/messageQueue/kafka.go
+2 -1
View File
@@ -17,6 +17,7 @@ limitations under the License.
package messageQueue package messageQueue
import ( import (
"fmt"
"io/ioutil" "io/ioutil"
"net/http" "net/http"
"os" "os"
@@ -214,7 +215,7 @@ func kafkaMsgHandler(kafka *Kafka, producer sarama.SyncProducer, trigger *fv1.Me
} }
if resp.StatusCode != 200 { if resp.StatusCode != 200 {
errorHandler(kafka.logger, trigger, producer, url, errorHandler(kafka.logger, trigger, producer, url,
errors.Wrapf(err, "request returned failure: %v", resp.StatusCode)) fmt.Errorf("request returned failure: %v", resp.StatusCode))
return false return false
} }
if len(trigger.Spec.ResponseTopic) > 0 { if len(trigger.Spec.ResponseTopic) > 0 {