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

add bower config #5

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

add bower config #5

wants to merge 1 commit into from

Conversation

pourquoi
Copy link

then you need to run

bower register meteorserver git://github.com/visitsb/meteorserver

@pourquoi
Copy link
Author

sorry it's bower register meteorserver git://github.com/visitsb/meteorserver.git

@visitsb
Copy link
Owner

visitsb commented Jul 14, 2014

@pourquoi thanks for the bower.json config. meteorserver is not yet ready as a node module, but there is plenty of scope (and interest) to do so. Your bower integration is definitely going to help in making that first step.

If I understand correctly, bower will only get the meteor.js but none of the server-side modules yet. Those are still left as a separate install. Only using bower will not get meteorserver up, and running.

Am I correct?

@pourquoi
Copy link
Author

yes, it will only expose the content of public_html

this is usefull if you serve those files with nginx/apache instead of meteor

@visitsb
Copy link
Owner

visitsb commented Jul 16, 2014

@pourquoi thanks for clarifying.

meteorserver needs to obey same origin policy when serving javascript files out of public_html. That's mainly the reason why meteorserver channel can initiate push messages to callers without throwing browsers into security alert.

Having any other webserver today is not straightforward, unless some bridge is established to proxy meteorserver behind the scenes.

What I'll do is hold onto your bower.json, and work something towards enabling that as a single bower configuration that can work right out-of-the-box.

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.

2 participants