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

Browser Window Opens but No Content #9

Closed
salcode opened this issue Jan 23, 2015 · 11 comments
Closed

Browser Window Opens but No Content #9

salcode opened this issue Jan 23, 2015 · 11 comments

Comments

@salcode
Copy link

salcode commented Jan 23, 2015

I'm running
livedown start ./README.md --open --verbose
which opens a new tab in Chrome which appears empty.

The url is
http://localhost:1337/

Sanity Check

less ./README.md displays the content of the file, leading me to believe my file path is correct

Setup

  • Chrome 39.0
  • OS X 10.9.5

Console Errors

In the Chrome developer console, I'm seeing

Failed to load resource: net::ERR_CONNECTION_REFUSED  http://localhost/socket.io/?EIO=3&transport=polling&t=1422024891424-0

This error is repeated multiple times.

@salcode
Copy link
Author

salcode commented Jan 23, 2015

Looking at the line

http://localhost/socket.io/?EIO=3&transport=polling&t=1422024891424-0

it appears the port (1337 in this example) is missing.

@mgrubb
Copy link

mgrubb commented Jan 26, 2015

I'm having the same exact problem as well. After digging through the code it appears that index.html is being sent without adding the port number in the io.connect() call. This is weird as this was working for me last week and going through the code I can't find anything obvious. However, I do have a patch f48be49 that fixes this.

@freethejazz
Copy link

I'm experiencing this as well (through the use of the vim plugin) and am a little wary of not knowing the cause. It seems like this broke for at least the three of us for no reason. I haven't updated the package or plugin and am curious about why it stopped working all of a sudden. Maybe it has to do with side effects relating an upgrade of one of the dependencies?

@freethejazz
Copy link

I think the use of ^'s in package.json is the problem. I just removed them all in my local copy of livedown and ran npm install to get the exact version numbers. Things are back to normal now. Rooting through now to figure out exactly which dependency is causing the issue.

@freethejazz
Copy link

So the culprit is socket.io version 1.3.2, which really just updates socket.io-client to it's version 1.3.2. @mgrubb, your patch probably addresses that exact issue, but I can't look at it at this moment.

In the mean time, my workaround is to specify the exact version number in the livedown package.json where it's installed. In my case:

$ cd ~/.nvm/v0.10.28/lib/node_modules/livedown
$ vim package.json

# edit the version of socket.io to be "1.3.1", save and quit

$ rm -rf node_modules
$ npm install

@mgrubb
Copy link

mgrubb commented Jan 27, 2015

I wish what I did was as elegant as that :)
I just hacked a template handler into server.js to substitute the url
into index.html.

Jonathan Freeman mailto:notifications@github.com
January 26, 2015 at 5:28 PM

So the culprit is |socket.io| version 1.3.2, which really just updates
|socket.io-client| to it's version 1.3.2
socketio/socket.io-client@ea9b2b7.
@mgrubb https://github.com/mgrubb, your patch probably addresses
that exact issue, but I can't look at it at this moment.

In the mean time, my workaround is to specify the exact version number
in the livedown |package.json| where it's installed. In my case:

|$ cd ~/.nvm/v0.10.28/lib/node_modules/livedown
$ vim package.json

edit the version of socket.io to be "1.3.1", save and quit

$ rm -rf node_modules
$ npm install
|


Reply to this email directly or view it on GitHub
#9 (comment).

@salcode
Copy link
Author

salcode commented Jan 29, 2015

@shime I see you've fixed this with #11

When you have a chance would you please tag a new release and push it to npm?

Thanks.

@shime
Copy link
Owner

shime commented Jan 29, 2015

Sure, thanks for the ping!

@shime shime closed this as completed Jan 29, 2015
@salcode
Copy link
Author

salcode commented Jan 30, 2015

@shime thanks for the quick response, I see 1.0.8 tagged here on GitHub but I'm still seeing 1.0.7 on
npm

@shime
Copy link
Owner

shime commented Jan 30, 2015

@salcode forgot to publish it on NPM again. It's up now, thanks!

@salcode
Copy link
Author

salcode commented Jan 30, 2015

@shime Awesome, thank you.

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

No branches or pull requests

4 participants