Skip to content
This repository has been archived by the owner on Jan 5, 2024. It is now read-only.

[FIX] Bypass cookie validation #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[FIX] Bypass cookie validation #23

wants to merge 1 commit into from

Conversation

sfeuga
Copy link

@sfeuga sfeuga commented Oct 18, 2017

Monkey patch for this issue: #19

#
###unless domains_match cookie_host, uri
### errors << 'Domain is inappropriate based on request URI hostname'
###end
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after #.

###
# MonkeyPatch
#
###unless domains_match cookie_host, uri
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space after #.

pre added a commit to smartlyio/cookiejar that referenced this pull request May 17, 2018
There has been an open issue in cookiejar Gem since 2014.
The project seems unmaintained, but it's used by the faye gem.

References:
* restforce/restforce#120
* dwaite#19
* dwaite#23
@ChaoticBoredom
Copy link

@dwaite Is this still maintained? This seems like a reasonable PR and is something our organization needs.

@markcatley
Copy link

FYI I just raised #35 that fixes this in a standards-compliant way.

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.

4 participants