Unmarshal error is an GET_INTERNAL_SERVER error

prebid · Nov 30, 2023 · 64d357e · 64d357e
1 parent 74f808b
commit 64d357e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/backends/ignite.go b/backends/ignite.go
@@ -190,7 +190,7 @@ func (ig *IgniteBackend) Get(ctx context.Context, key string) (string, error) {
 	igniteResponse := getResponse{}
 
 	if unmarshalErr := json.Unmarshal(responseBytes, &igniteResponse); unmarshalErr != nil {
-		return "", fmt.Errorf("Unmarshal response error: %s; Response body: %s", unmarshalErr.Error(), string(responseBytes))
+		return "", utils.NewPBCError(utils.GET_INTERNAL_SERVER, fmt.Sprintf("Ignite response unmarshal error: %s; Response body: %s", unmarshalErr.Error(), string(responseBytes)))
 	}
 
 	// Validate response

diff --git a/backends/ignite_test.go b/backends/ignite_test.go
@@ -441,7 +441,7 @@ func TestIgniteGet(t *testing.T) {
 				igniteError:    nil,
 			},
 			expected: testOutput{
-				err: errors.New("Unmarshal response error: invalid character 'm' looking for beginning of value; Response body: malformed"),
+				err: utils.NewPBCError(utils.GET_INTERNAL_SERVER, "Ignite response unmarshal error: invalid character 'm' looking for beginning of value; Response body: malformed"),
 			},
 		},
 		{

diff --git a/config.yaml b/config.yaml
@@ -11,7 +11,7 @@ request_limits:
   max_num_values: 10
   max_ttl_seconds: 3600
 backend:
-  type: "memory" # Switch to be "aerospike", "cassandra", "memcache" or "redis" for production.
+  type: "memory" # Switch to be "aerospike", "cassandra", "memcache", "ignite" or "redis" for production.
   aerospike:
     hosts: [ "aerospike.prebid.com" ]
     port: 3000