veliovgroup / jazeee-meteor-spiderable

Fork of Meteor Spiderable with longer timeout, caching, better server handling
https://atmospherejs.com/jazeee/spiderable-longer-timeout
33 stars 9 forks source link

Meteor 1.4 + React: 204 No content #39

Closed dnish closed 7 years ago

dnish commented 7 years ago

Hey, I'm having a issue with your package on my local dev machine. I'm using Meteor 1.4 with React and getting 204 No content after requesting http://localhost:3000/?_escaped_fragment_= Spiderable debug doesn't show anything. Does this package still work with 1.4?

dr-dimitru commented 7 years ago

@dnish It's should be http://localhost:3000/?_escaped_fragment_= (two underscores is missed)

dnish commented 7 years ago

@dr-dimitru Yeah, I've used the second one. GitHub removes it when it is wrapped into code. After calling it nothing happens on the page, instead I can see the 204 response in my network console.

dr-dimitru commented 7 years ago

@dnish drop the cache and try again

dnish commented 7 years ago

I've just tried, even in privacy mode I get a 204 if I add the escaped fragment parameter. All other query parameters work fine.

dr-dimitru commented 7 years ago

I've meant cache of this lib, the mongo collection

dnish commented 7 years ago

The SpiderableCacheCollection is empty

dr-dimitru commented 7 years ago

Use Spiderable.debug = true to see what is going on

dnish commented 7 years ago

I'm getting this message in my console:

Meteor application returned empty response
dr-dimitru commented 7 years ago

@dnish Phantomjs binary is installed on your machine?

dnish commented 7 years ago

Yep, I'll try now to install it again. Maybe it doesn't work right because of the MacOS upgrade.

dr-dimitru commented 7 years ago

Better to compile it locally

dnish commented 7 years ago

Mh okay, reinstalling solved the issue. But now I get only an empty page without the rendered React components in it. Pages is saved within SpiderableCacheCollection.

dr-dimitru commented 7 years ago

Hello @dnish ,

I know it's kind of late. But we just updated library to work with latest Meteor and we have all new codebase. Could you test it on your end?

dr-dimitru commented 7 years ago

Closed due to silence at issue owner end. Feel free to reopen it in case if the issue still persists on your end.