Cookie string doesn't get parsed ok by Rook

Ouverte
#39 2 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

É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
r
Domaine
backend

Piste de recherche

Start by reproducing the provided app with httpuv::runServer and compare its cookie behavior with Rook::Rhttpd$new(). Focus on the interaction between Request$cookies() and req$HTTP_COOKIE; done means newly set cookies are recognized after refreshing under runServer, matching the working Rhttpd path.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

This seems to be a Rook/httpuv interaction. Using Rook::Rhttpd, things work OK. For me: new cookies are not recognized by Rook::Request$cookies() when I refresh, even though they are visible in the HTTP_COOKIE element of the environment.

R 3.1.1, Rook 1.1.1, httpuv 1.3.2.

Test case:

library(httpuv)
library(Rook)
ctr <- 1
app <- list(call=function(req) {
  rr <- Request$new(req)
  res <- Response$new()
  res$set_cookie(paste0("foo", ctr), paste0("bar", ctr))
  res$write("<html><body><p>Hit refresh to see the name of the cookie I just set... it should be: foo")
  res$write(ctr)
  res$write("</p>")
  ctr <<- ctr + sample(1:1000, 1)

  res$write("<p>Cookie names seen by rr$cookies():</p>")
  res$write(paste(names(rr$cookies()), collapse="<br>"))
  res$write("<p>cookie string from request env:</p>")
  res$write(req$HTTP_COOKIE)
  res$write("</body></html>")

  return(res$finish())
})
# won't work:
runServer("127.0.0.1", 4444, app)
# works:
rh <- Rhttpd$new()
rh$launch(app$call, "cookie")
Langage dominant
C
Étoiles
252
Forks
87
Métriques de merge des PR
Aucune PR mergée en 30 j

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de rstudio/httpuv

Toutes les issues de rstudio/httpuv

Issues similaires

Plus d'issues C

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.