search

gadicc / meteor-headers

Access HTTP headers on both server and client. Client IP with proxy support.
https://atmospherejs.com/gadicohen/headers
GNU Lesser General Public License v3.0
61 stars 21 forks source link

Serious Error , Site stops running #35

Closed jamesfebin closed 9 years ago

jamesfebin commented 9 years ago

Error: url must be a relative URL: http://www.mafengwo.com/ at _.extend.classify (packages/routepolicy/routepolicy.js:103) at Object.Inject.appUrl (packages/meteorhacks:inject-initial/lib/inject-core.js:43) at Object.Package as handle at next (/home/timekeeper/bundle/programs/server/npm/webapp/node_modules/connect/lib/proto.js:190:15) at next (/home/timekeeper/bundle/programs/server/npm/webapp/node_modules/connect/lib/proto.js:165:78) at Function.app.handle (/home/timekeeper/bundle/programs/server/npm/webapp/node_modules/connect/lib/proto.js:198:3) at Object.fn as handle at next (/home/timekeeper/bundle/programs/server/npm/webapp/node_modules/connect/lib/proto.js:190:15) at Object.WebAppInternals.staticFilesMiddleware (packages/webapp/webapp_server.js:400) at packages/webapp/webapp_server.js:651

gadicc commented 9 years ago

Hey, can you tell me what release of meteor you're using and also paste .meteor/versions? (or just mention the versions of routepolicy, webapp, gadicohen:headers and meteorhacks:inject-initial)

Is this a new error or as soon as you installed the package, your site stopped working?

jamesfebin commented 9 years ago

The error existed even before i upgraded. This occasionally occurs when some bot is trying to access the site illegally I am using Meteor 1.0

Verson

routepolicy@1.0.2 webapp@1.1.4 gadicohen:headers@0.0.26

This error was seen in webapp. But they fixed it https://github.com/meteor/meteor/pull/2393

Regards,

Febin John James Co-Founder

On Sat, Nov 8, 2014 at 11:47 AM, Gadi Cohen notifications@github.com wrote:

Hey, can you tell me what release of meteor you're using and also paste .meteor/versions? (or just mention the versions of routepolicy, webapp, and gadicohen:headers)

Is this a new error or as soon as you installed the package, your site stopped working?

— Reply to this email directly or view it on GitHub https://github.com/gadicc/meteor-headers/issues/35#issuecomment-62247569 .

gadicc commented 9 years ago

Hey, thanks for the ref to the meteor issue, super helpful! Unfortunately what this means though is that the problem isn't with headers/inject-initial, it's with Meteor itself, and beyond our control.

Note that that PR is still open and not yet merged, and the issue it links to is still open too. Seems like quite a serious error which really should get fixed :)

gadicc commented 9 years ago

Hey, closing and marking as invalid, just because the issue is with Meteor and not the headers package. Hope there has been some progress with this since November, any updates are welcome.