6648
62%
master: %

Ran 20 May 2019 05:34PM UTC

Jobs 1

Files 106

Run time 7s

Badge

Embed ▾

pending completion

Build # 6648

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

10552 of 16926 relevant lines covered (62.34%)

0.7 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	6648.1 (LATEST_GO=true)	20 May 2019 05:34PM UTC	0	62.34	Travis Job 6648.1

Source Files on build 6648

Detailed source file information is not available for this build.

TykTechnologies / tyk / 6648
62%
master: %

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 6648

TykTechnologies / tyk / 6648 62% master: %

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 6648

TykTechnologies / tyk / 6648
62%
master: %

README BADGES
x