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

Add option to read multiple cookies with same name #108

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

SachinShekhar
Copy link

Fixes #60

HISTORY.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
test/parse.js Show resolved Hide resolved
@SachinShekhar
Copy link
Author

@dougwilson Done!

@oliverhausler
Copy link

@dougwilson Any reason why @SachinShekhar 's PR is not merged. We would need that functionality. It's a normal thing that may happen so the library should be able to handle it. Seems Sachin has fixed the addressed issues.

@Abdelaziz18003
Copy link

@SachinShekhar @dougwilson any news about this PR? We would really love if this feature got addressed and merged. Thanks so much for your time and effort.

@Abdelaziz18003
Copy link

@dougwilson I've managed to get this PR working properly with the latest version (v0.5.0) in a separate project. I would like to know if there is anything I can do to help get that feature added. I can make a separate PR if needed. Thanks.

@rfernandezapML
Copy link

@dougwilson is there anything I can do to have this PR merged? We need this feature 🙏

@jshttp jshttp deleted a comment from tpae Jan 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reading duplicated cookies
5 participants