Do not supply a default content type.
Une pull request liée a déjà été fusionnée.
- #207 par @hsbt — fusionnée
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- ruby
- Domaine
- networking
Piste de recherche
Commencez par supply_default_content_type dans lib/net/http/generic_request.rb vers la ligne 263, puis comparez son comportement aux recommandations de la RFC 7231 citées dans l’issue. La modification est terminée lorsque Net::HTTP ne fournit plus de Content-Type supposé pour un body inconnu, y compris le cas d’utilisation de type AWS décrit.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Hello.
I would like to know if you would welcome a change to remove suppy_default_content_type.
Checking RFC for Content-Type: https://datatracker.ietf.org/doc/html/rfc7231#section-3.1.1.5
A sender that generates a message containing a payload body SHOULD
generate a Content-Type header field in that message unless the
intended media type of the enclosed representation is unknown to the
sender. If a Content-Type header field is not present, the recipient
MAY either assume a media type of "application/octet-stream"
([RFC2046], Section 4.5.1) or examine the data to determine its type.
Based on the RFC, I think Net::HTTP behavior is incorrect - assuming a default content type is not correct because the media type is not known unless Net::HTTP reads/inspects the body. The receiver may assume octet-stream unless provided.
This has caused issues with AWS services, where content type may be a modeled API parameter in REST services, like with S3 where you can specify the content type of an object. Currently we work around this with a patch:
Thread.current[:net_http_skip_default_content_type] = true
def self.apply!
Net::HTTPGenericRequest.prepend(PatchDefaultContentType)
end
module PatchDefaultContentType
def supply_default_content_type
return if Thread.current[:net_http_skip_default_content_type]
super
end
end
- Langage dominant
- Ruby
- Étoiles
- 148
- Forks
- 95
- Merge moyen
- 3 h 34 min
- PR mergées (30 j)
- 2
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de ruby/net-http
-
Empty DOM attribute crashes tokenizerPeut-être pris @jeremyevans l’a pris il y a 86 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
Unquote and unescape Content-Type: charset [RFC7230]Peut-être pris @jeremyevans l’a pris il y a 86 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 38/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
Toutes les issues de ruby/net-http
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
rubys/roundhouse#444 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
yegor256/rubocop-elegant#92 ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
ecosyste-ms/awesome#812 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 63/100
slovensko-digital/autogram-portal#383 ·
Les mainteneurs répondent en général sous 1 jour