Dancer::Serializer::Mutable fixes #996

wants to merge 2 commits into


None yet
2 participants

aldot commented Feb 10, 2014


Attached are two patches to fix two distinct bugs in the Mutable serializer.

Please install.

TIA && cheers,

aldot added some commits Feb 10, 2014

Serializer::Mutable: Fix content-type with charset
Previously using a header of
Content-Type: text/xml; charset=utf-8
failed to load the correct serializer, json was used as a fallback.

Signed-off-by: Bernhard Reutner-Fischer <>
Serializer::Mutable: Fix reponse content-type
curl -X PUT -k -i \
 -H 'Accept: application/json; charset=UTF-8' \
 -H 'Content-Type: text/xml; charset=UTF-8' \
 -d '<r><elt1>content1</elt1></r>'

failed to find the correct RESPONSE serializer.
Content-Type is the format of the input while Accept is the format for
the response data (output).

Signed-off-by: Bernhard Reutner-Fischer <>

@yanick yanick closed this in 363467e Mar 19, 2014

This comment has been minimized.

Show comment Hide comment

yanick Mar 19, 2014


Merged, thanks!


yanick commented Mar 19, 2014

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment