BUG/MINOR: redispatch: allow interval of 0

Author: mjuraga

.aspell.yml

@@ -128,6 +128,7 @@ allowed:
   - readme
   - recursivity
   - recv
+  - redispatch
   - redoc
   - reimplement
   - releaser

configuration/configuration.go

@@ -995,9 +995,8 @@ func (s *SectionParser) redispatch() interface{} {
 		return nil
 	}
 	d := data.(*types.OptionRedispatch)
-	br := &models.Redispatch{}
-	if d.Interval != nil {
-		br.Interval = *d.Interval
+	br := &models.Redispatch{
+		Interval: d.Interval,
 	}
 	if d.NoOption {
 		d := "disabled"
@@ -2436,15 +2435,12 @@ func (s *SectionObject) redispatch(field reflect.Value) error {
 			return misc.CreateTypeAssertError("option redispatch")
 		}
 		d := &types.OptionRedispatch{
-			Interval: &br.Interval,
+			Interval: br.Interval,
 			NoOption: false,
 		}
 		if *br.Enabled == "disabled" {
 			d.NoOption = true
 		}
-		if br.Interval == 0 {
-			d = nil
-		}
 		if err := s.set("option redispatch", d); err != nil {
 			return err
 		}

models/redispatch.go

@@ -10589,7 +10589,7 @@ definitions:
         type: string
       interval:
         type: integer
-        x-nullable: false
+        x-nullable: true
     required:
       - enabled
     type: object

models/redispatch_compare.go

@@ -151,7 +151,7 @@ redispatch:
       enum: [enabled, disabled]
     interval:
       type: integer
-      x-nullable: false
+      x-nullable: true
 errorfile:
   type: object
   x-display-name: Error File

models/redispatch_compare_test.go

@@ -222,7 +222,7 @@ func TestCreateEditDeleteBackend(t *testing.T) {
 			Disabled: true,
 			Redispatch: &models.Redispatch{
 				Enabled:  misc.StringP("enabled"),
-				Interval: 0,
+				Interval: misc.Int64P(0),
 			},
 		},
 	}
@@ -239,9 +239,6 @@ func TestCreateEditDeleteBackend(t *testing.T) {
 		t.Error(err.Error())
 	}
 
-	// A redispatch with a zero interval is automatically removed from configuration.
-	b.Redispatch = nil
-
 	var givenJSONB []byte
 	givenJSONB, err = b.MarshalBinary()
 	if err != nil {