1
62%
master: %

Ran 20 May 2019 05:34PM UTC

Files 106

Run time 6s

Badge

Embed ▾

Committed 20 May 2019 05:27PM UTC coverage: 62.342% (-0.01%) from 62.353%

Job # LATEST_GO=true

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`.

Coverage Stats

10552 of 16926 relevant lines covered (62.34%)

0.7 hits per line

TykTechnologies / tyk / 6648 / 1
62%
master: %

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 6648.1 (LATEST_GO=true)

TykTechnologies / tyk / 6648 / 1 62% master: %

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 6648.1 (LATEST_GO=true)

TykTechnologies / tyk / 6648 / 1
62%
master: %

README BADGES
x