akka/akka-http

Marshaller.withFixedContentType doesn't seem to work

Ouverte

#1 077 ouverte le 2 mai 2017

Voir sur GitHubGuide contributeur
 (4 commentaires) (0 réaction) (0 personne assignée)Scala (598 forks)batch import
1 - triagedbughelp wantedt:docst:marshalling

Métriques du dépôt

Stars
 (1 311 étoiles)
Métriques de merge PR
 (Merge moyen 1j 10h) (2 PRs mergées en 30 j)

Description

When I create a ToEntityMarshaller for a type with Marshaller.withFixedContentType, as so:

  case class SomeType2(content: String)

  implicit val st2AsCsv: ToEntityMarshaller[SomeType2] =
    Marshaller.withFixedContentType(ContentTypes.`text/csv(UTF-8)`)(_.content)

the response is still returning Content-Type: text/plain; charset=UTF-8. Using Marshaller.StringMarshaller.wrap works, though.

  case class SomeType1(content: String)

  implicit val st1AsCsv: ToEntityMarshaller[SomeType1] =
    Marshaller.StringMarshaller.wrap(MediaTypes.`text/csv`)(_.content)

Here's a minimal example to reproduce: https://gist.github.com/chadselph/9149bab77c7aebfcea24bc15c9b82495

I've only tested this on 10.0.5, am I doing something wrong or is this a bug?

Guide contributeur

Direction de recherche
Inspectez l'implémentation de Marshaller.withFixedContentType et tracez pourquoi elle ne remplace pas le type de contenu comme prévu. Comparez avec Marshaller.StringMarshaller.wrap pour identifier la différence.
Stack technique
scala
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Scala
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60