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

window.location.hostname to window.location.host #120

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

Conversation

amol9
Copy link

@amol9 amol9 commented Jul 20, 2015

Changed window.location.hostname to window.location.host so that port number is included in the url.

Fixes the following type of error:

phantom stdout: console msg:XMLHttpRequest cannot load http://127.0.0.1/socket.io/?EIO=3&>transport=polling&t=1437374191861-0. Origin http://127.0.0.1:44550 is not allowed by Access-Control-Allow-Origin.

number is included in the url.

Fixes the following type of error:
>>
phantom stdout: console msg:XMLHttpRequest cannot load http://127.0.0.1/socket.io/?EIO=3&transport=polling&t=1437374191861-0. Origin http://127.0.0.1:44550 is not allowed by Access-Control-Allow-Origin.
<<
@avegancafe
Copy link

Life saver, +1

@pvsrivathsa
Copy link

Please accept this PR as it is a blocker. Not able to use the phantomjs module at all

@TiagoJacobs
Copy link

+1, can you merge, please?

@stianl
Copy link

stianl commented Nov 9, 2015

Is this going to be fixed soon?

@khaledosman
Copy link

+1, version 2 doesn't work and version 1 has this issue..

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.

6 participants