check for nil for evt.Response (#1007)

* check for nil for evt.Response * bump version to 1.5.1-rc2 * make docsgen
filecoin-project · Nov 24, 2022 · 2c5604e · 2c5604e
1 parent edab3de
commit 2c5604e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/build/openrpc/boost.json.gz b/build/openrpc/boost.json.gz
diff --git a/build/version.go b/build/version.go
@@ -2,7 +2,7 @@ package build
 
 var CurrentCommit string
 
-const BuildVersion = "1.5.1-rc1"
+const BuildVersion = "1.5.1-rc2"
 
 func UserVersion() string {
 	return BuildVersion + CurrentCommit

diff --git a/retrievalmarket/rtvllog/retrieval_log.go b/retrievalmarket/rtvllog/retrieval_log.go
@@ -48,8 +48,6 @@ func (r *RetrievalLog) OnQueryEvent(evt retrievalmarket.ProviderQueryEvent) {
 // Storage Provider validates the request (eg checking its parameters for
 // validity, checking for acceptance against the retrieval filter, etc)
 func (r *RetrievalLog) OnValidationEvent(evt retrievalmarket.ProviderValidationEvent) {
-	log.Debugw("validation-event", "id", evt.Response.ID, "status", evt.Response.Status, "msg", evt.Response.Message, "err", evt.Error)
-
 	// Ignore ErrPause and ErrResume because they are signalling errors, not
 	// actual errors because of incorrect behaviour.
 	if evt.Error == nil || evt.Error == datatransfer.ErrPause || evt.Error == datatransfer.ErrResume {
@@ -64,6 +62,8 @@ func (r *RetrievalLog) OnValidationEvent(evt retrievalmarket.ProviderValidationE
 		Message:    evt.Error.Error(),
 	}
 	if evt.Response != nil {
+		log.Debugw("validation-event", "id", evt.Response.ID, "status", evt.Response.Status, "msg", evt.Response.Message, "err", evt.Error)
+
 		st.DealID = evt.Response.ID
 		st.Status = evt.Response.Status.String()
 		if evt.Response.Message != "" {