ruby-grape / grape
100%
master: 100%

LAST BUILD BRANCH: dblock/automate-gem-release
DEFAULT BRANCH: master
Repo Added
Files 179
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH fix/negotiated-format-fallback
branch: fix/negotiated-format-fallback
CHANGE BRANCH
x

coverage: 99.966% (-0.03%) from 100.0%
36440809203

Pull #3015

github

ericproulx 
Answer in the default format when a negotiated format cannot render the body

On an API that pins no format, content negotiation can pick a format
the endpoint's body cannot be rendered in. A browser's Accept header
(text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8)
negotiates XML at q=0.9, and axios's (application/json, text/plain, */*)
ties at q=1 and Rack picks the first registered type, XML. The XML
formatter cannot render a String, so every endpoint returning one
answered those clients with a 500 ("cannot convert String to xml"), as
did `.xml` and `?format=xml`.

When the negotiated formatter raises InvalidFormatter, the Formatter now
renders the body in the default format, with its content type. It does
so only while the format is the client's alone: an API that pins a
format, an endpoint that set a Content-Type, or one that switched
`api_format` still answers 500, since a body that does not fit a choice
the API made is the API's error. If the default format cannot render
the body either, the 500 stands.

`api_spec.rb`'s "sets content type for xml" requested `/foo.xml` from an
endpoint returning a String, so it was checking the content type of that
500; it now requests a body XML can render.

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Pull Request #3015: Answer in the default format when a negotiated format cannot render the body

1508 of 1509 branches covered (99.93%)

Branch coverage included in aggregate %.

18 of 18 new or added lines in 1 file covered. (100.0%)

4349 of 4350 relevant lines covered (99.98%)

29621.42 hits per line

Relevant lines Covered
4350 RELEVANT LINES 4349 COVERED LINES
29621.42 HITS PER LINE
Source Files on fix/negotiated-format-fallback

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
36440809203 fix/negotiated-format-fallback Answer in the default format when a negotiated format cannot render the body On an API that pins no format, content negotiation can pick a format the endpoint's body cannot be rendered in. A browser's Accept header (text/html,application/xhtml+xm... Pull #3015 ericproulx github
99.97
36438967994 fix/negotiated-format-fallback Answer in the default format when a negotiated format cannot render the body On an API that pins no format, content negotiation can pick a format the endpoint's body cannot be rendered in. A browser's Accept header (text/html,application/xhtml+xm... Pull #3015 ericproulx github
99.98
36438044714 fix/negotiated-format-fallback Answer in the default format when a negotiated format cannot render the body On an API that pins no format, content negotiation can pick a format the endpoint's body cannot be rendered in. A browser's Accept header (text/html,application/xhtml+xm... Pull #3015 ericproulx github
99.98
36408649305 fix/negotiated-format-fallback Answer in the default format when a negotiated format cannot render the body On an API that pins no format, content negotiation can pick a format the endpoint's body cannot be rendered in. A browser's Accept header (text/html,application/xhtml+xm... Pull #3015 ericproulx github
100.0
36251478436 fix/negotiated-format-fallback Answer in the default format when a negotiated format cannot render the body On an API that pins no format, content negotiation can pick a format the endpoint's body cannot be rendered in. A browser's Accept header (text/html,application/xhtml+xm... Pull #3015 ericproulx github
100.0
36251466585 fix/negotiated-format-fallback Answer in the default format when a negotiated format cannot render the body On an API that pins no format, content negotiation can pick a format the endpoint's body cannot be rendered in. A browser's Accept header (text/html,application/xhtml+xm... push ericproulx github
100.0
See All Builds (4537)