Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

get method doesn't works correctly #19

Open
giolf opened this issue May 14, 2017 · 7 comments
Open

get method doesn't works correctly #19

giolf opened this issue May 14, 2017 · 7 comments

Comments

@giolf
Copy link

giolf commented May 14, 2017

If i add a new cookie by the setmethod then with the getmethod i can fetch it.
BUT if i use the getmethod to fetch a cookie that already exist (so i din't use the setmethod). In that case the getmethod doesnt find anything!

@pqt
Copy link

pqt commented Jul 11, 2017

Was just about to come here and report the same thing. Is this actually a known issue ?

@giolf
Copy link
Author

giolf commented Jul 12, 2017

@austinpaquette, unfortunately, I don't think so.

@stevepop
Copy link

Hi guys I have experienced the same thing too, and came here to see its been logged as an issue. From the silence from the Author, looks like no solution in sight yet.

@ivolimasilva
Copy link

Had this same issue when setting cookies from server. After searching I realised the cookies were set with HttpOnly flag as true, since it was default. After changing it to false, I was able to fetch it using the get method.

Be sure the same isn't happening to you guys.

@stevepop
Copy link

@ivolimasilva, Fixed mine as well by setting HttpOnly to false.

@AustinFelipe
Copy link

It works for me as well. It should be closed. 😌

@OctaneInteractive
Copy link

I'm still having the problem, having tried two different packages.

app.use(
  session({
    name: 'sessionName',
    secret: process.env.SESSION_SECRET,
    saveUninitialized: true,
    resave: false,
    httpOnly: false,
    cookie: {
      secure: false
    }
  })
)

Any ideas?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants