thedevsaddam / govalidator

89%

master: 89%

LAST BUILD ON BRANCH v1.9.5

branch: v1.9.5

CHANGE BRANCH

x

Reset

• v1.9.5
• dev
• feature/file-validation
• fix/date-validation
• fix/digits-gt-13
• fix/numeric-field
• master
• revert-19-issue-18
• v1.2
• v1.3
• v1.4
• v1.5
• v1.6
• v1.8
• v1.9
• v1.9.1
• v1.9.10
• v1.9.2
• v1.9.3
• v1.9.4
• v1.9.6
• v1.9.7
• v1.9.8
• v1.9.9
• v2

Build # 197

Build Type

push

travis-ci

Committed by web-flow

Commit Message

Merge pull request #51 from RexSkz/rex/prevent-panic-caused-by-null-value

If the request body is a JSON like `{"test":null}`, govalidator will panic when executing `traverseMap`.

```go
var m = make(map[string]interface{})
json.Unmarshal([]byte(`{"test":null}`), &m)
```

Since `m["test"]` is `nil`, `reflect.TypeOf(v)` will panic. This PR fixes the bug by dropping null values in json.

Run Details

1331 of 1498 relevant lines covered (88.85%)

174.34 hits per line