Skip to content
This repository has been archived by the owner on Apr 27, 2020. It is now read-only.

Add cookie support #24

Closed
wants to merge 1 commit into from
Closed

Conversation

rogeriochaves
Copy link

The user just need to have write permissions for the cookie.txt file

The user just need to have write permissions for the cookie.txt file
@joshdick
Copy link
Owner

I've thought about using CURLOPT_COOKIEJAR before, but I don't believe this is a good solution for multiple simultaneous users since they will all be sharing the same set of cookies.

As it is, this code may not work as intended anyway -- you may need to use CURLOPT_COOKIEFILE in conjunction with CURLOPT_COOKIEJAR (COOKIEJAR is the write location, while COOKIEFILE is the read location) but I haven't tried this myself.

@joshdick joshdick closed this Nov 17, 2015
@rogeriochaves
Copy link
Author

@joshdick yeah, I didn't thought about the multiple simultaneous users case, thanks for noticing

@lesander
Copy link

PHP Session-specific cookies?

@joshdick
Copy link
Owner

joshdick commented Dec 31, 2015

@lesander This is mostly a note for myself :) Even with session-specific cookies, special care would need to be taken to make sure proxied sites don't unintentionally receive each others' cookies, even when originating from a single user, since the users' browser would see each different site as having identical origins (the proxy).

This StackOverflow answer is relevant to this idea.

@joshdick
Copy link
Owner

I just opened #69 for cookie support. It works in very basic testing but isn't production ready (yet); use it at your own risk!

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

Successfully merging this pull request may close these issues.

3 participants