Merge pull request #28 from kpetremann/panic_not_built_device

fix: panic when user requests non built config
criteo · Jan 11, 2024 · db05db1 · db05db1
2 parents 7c52799 + 639abda
commit db05db1
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/internal/convertor/device/serializer.go b/internal/convertor/device/serializer.go
@@ -88,6 +88,10 @@ func (s *SafeRepository) GetDeviceOpenConfigJSON(hostname string) ([]byte, error
 	defer s.mutex.Unlock()
 
 	if dev, ok := s.devices[hostname]; ok {
+		if dev == nil {
+			return []byte(emptyJSON), errors.New("build failed for this device")
+		}
+
 		var config json.RawMessage
 		var err error
 		out := bytes.NewBuffer(nil)