feat #582: fix date/date-time string formats

Author: cboitel

openapi3/schema_formats.go

@@ -172,12 +172,11 @@ func init() {
 	// The pattern supports base64 and b./ase64url. Padding ('=') is supported.
 	DefineStringFormat("byte", `(^$|^[a-zA-Z0-9+/\-_]*=*$)`)
 
-	// date
-	DefineStringFormat("date", `^[0-9]{4}-(0[0-9]|10|11|12)-([0-2][0-9]|30|31)$`)
-
-	// date-time
-	DefineStringFormat("date-time", `^[0-9]{4}-(0[0-9]|10|11|12)-([0-2][0-9]|30|31)T[0-9]{2}:[0-9]{2}:[0-9]{2}(\.[0-9]+)?(Z|(\+|-)[0-9]{2}:[0-9]{2})?$`)
+	// defined as full-date in https://www.rfc-editor.org/rfc/rfc3339#section-5.6
+	DefineStringFormat("date", `^[0-9]{4}-(0[1-9]|1[0-2])-(0[1-9]|[12][0-9]|30|31)$`)
 
+	// defined as date-time in https://www.rfc-editor.org/rfc/rfc3339#section-5.6
+	DefineStringFormat("date-time", `^[0-9]{4}-(0[1-9]|1[012])-(0[1-9]|[12][0-9]|3[01])T(23:59:60|(([01][0-9]|2[0-3])(:[0-5][0-9]){2}))(\.[0-9]+)?(Z|(\+|-)[0-9]{2}:[0-9]{2})?$`)
 }
 
 // DefineIPv4Format opts in ipv4 format validation on top of OAS 3 spec

openapi3/schema_test.go

@@ -364,7 +364,30 @@ var schemaExamples = []schemaExample{
 			"56F5Bff4-Z4B6-48E6-a10D-B6CF66A83B04",
 		},
 	},
-
+	{
+		Title:  "STRING: format 'date'",
+		Schema: NewDateSchema(),
+		Serialization: map[string]interface{}{
+			"type":   "string",
+			"format": "date",
+		},
+		AllValid: []interface{}{
+			"2017-12-31",
+			"2017-01-01",
+		},
+		AllInvalid: []interface{}{
+			nil,
+			3.14,
+			"2017-12-00",
+			"2017-12-32",
+			"2017-13-01",
+			"2017-00-31",
+			"99-09-09",
+			"2017-01-00",
+			"2017-01-32",
+			"2017-01-40",
+		},
+	},
 	{
 		Title:  "STRING: format 'date-time'",
 		Schema: NewDateTimeSchema(),
@@ -379,6 +402,7 @@ var schemaExamples = []schemaExample{
 			"2017-12-31T11:59:59+11:30",
 			"2017-12-31T11:59:59.999+11:30",
 			"2017-12-31T11:59:59.999Z",
+			"2017-12-31T23:59:60", // leap second
 		},
 		AllInvalid: []interface{}{
 			nil,
@@ -387,6 +411,12 @@ var schemaExamples = []schemaExample{
 			"2017-12-31T11:59:59\n",
 			"2017-12-31T11:59:59.+11:30",
 			"2017-12-31T11:59:59.Z",
+			"2017-12-00T11:59:59.Z",
+			"2017-12-32T11:59:59.Z",
+			"2017-12-40T11:59:59.Z",
+			"2017-12-00T11:59:59",
+			"2017-12-31T11:59:60",
+			"99-09-09T11:59:59",
 		},
 	},