Skip to content

Support for alternate cookie domain#1

Open
rhubarbselleven wants to merge 3 commits intovalpo:masterfrom
rhubarbselleven:master
Open

Support for alternate cookie domain#1
rhubarbselleven wants to merge 3 commits intovalpo:masterfrom
rhubarbselleven:master

Conversation

@rhubarbselleven
Copy link
Contributor

Hi There,

I must say that your plugin has been handy. Here is a commit which includes support for an alternate cookie domain.

Eg: if the webserver is operating under a different domain to crowd.

@valpo
Copy link
Owner

valpo commented Mar 20, 2013

Hi,

sorry for the very late reply. I've merged the alternate domain stuff. The other commits contain a lot of format changes (removed lines, other indents aso...) so it is hard to guess what really changed...

@rhubarbselleven
Copy link
Contributor Author

Ugh, sorry about that, and my bad too for doing a pull request from my master branch.

The 2nd key change is that the request headers also include:

'Connection' => 'close'

Otherwise Joomla will run out of outgoing HTTP connections.

valpo pushed a commit that referenced this pull request Jun 21, 2018
Joomla 3 Changes and Group map supports multiple Joomla group ids per crowd group
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

Successfully merging this pull request may close these issues.

2 participants