6332

Ran 09 Mar 2019 11:43AM UTC

Jobs 1

Files 108

Run time 10s

Badge

Embed ▾

pending completion

Build # 6332

Build Type

push

travis-ci

Committed by

buger

Commit Message

res_handler_transform: make sure bodyData is available on XML errors (#2133)

For #2016.

I was able to reproduce using the instructions described in the ticket. The issue occurs at this point:

```go
	bodyData := make(map[string]interface{})
	switch tmeta.TemplateData.Input {
	case apidef.RequestXML:
		if len(body) == 0 {
			body = []byte("<_/>")
		}

		mxj.XmlCharsetReader = WrappedCharsetReader
		var err error

		bodyData, err := mxj.NewMapXml(body)
		if err != nil {
			logger.WithError(err).Error("Error unmarshalling XML")
		}
```

When `mxj.NewMapXml` returns an error, `bodyData` is not properly set and further references to it fail (like setting context vars using `bodyData["_tyk_context"] = ctxGetData(req)`).

This approach avoids replacing `bodyData`.

Run Details

10509 of 16869 relevant lines covered (62.3%)

0.69 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
2	6332.2 (LATEST_GO=true)	09 Mar 2019 11:43AM UTC	0	62.3	Travis Job 6332.2

Source Files on build 6332

Detailed source file information is not available for this build.

TykTechnologies / tyk / 6332

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 6332

TykTechnologies / tyk / 6332

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 6332

README BADGES
x