akka/akka-http

HTTP2: Respect SETTINGS_ENABLE_PUSH

Ouverte

#775 ouverte le 16 janv. 2017

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)Scala (598 forks)batch import
1 - triagedhelp wantedt:http2

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

SETTINGS_ENABLE_PUSH (0x2): This setting can be used to disable server push (Section 8.2).

  • An endpoint MUST NOT send a PUSH_PROMISE frame if it receives this parameter set to a value of 0.
  • (client) An endpoint that has both set this parameter to 0 and had it acknowledged MUST treat the receipt of a PUSH_PROMISE frame as a connection error (Section 5.4.1) of type PROTOCOL_ERROR.

The initial value is 1, which indicates that server push is permitted. Any value other than 0 or 1 MUST be treated as a connection error (Section 5.4.1) of type PROTOCOL_ERROR.

Guide contributeur

Direction de recherche
Implémentez la logique pour respecter SETTINGS ENABLE PUSH du pair distant : lors de la réception d'une trame SETTINGS avec ENABLE PUSH=0, le point de terminaison ne doit pas envoyer de trames PUSH PROMISE et doit traiter toute réception de PUSH PROMISE comme une erreur de connexion. Localisez l'implémentation HTTP/2 dans akka http et ajoutez les vérifications d'état et la gestion des erreurs nécessaires.
Stack technique
scala
Domaine
backendapi
Type d'issue
Fonctionnalité
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.
Une demi journée
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.
Plutôt claire
Prérequis
ScalaHTTP/2
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40