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

Example app should not use insecure/autopublish #1

Open
fusepilot opened this issue Dec 4, 2014 · 1 comment
Open

Example app should not use insecure/autopublish #1

fusepilot opened this issue Dec 4, 2014 · 1 comment

Comments

@fusepilot
Copy link

Any person visiting the cart can explore its entire history by running Cart.Items.find().fetch() in the console and iterate through the array of items. It's probably best to hide this data from the public.

@nate-strauser
Copy link
Owner

the package has tightly controlled publications https://github.com/nate-strauser/meteor-cart/blob/master/lib/server/publications.js which only send items attached to the userid or deviceid - the example app does have the insecure and autopublish packages still on though, which is likely what you are seeing

i'll update the example app shortly, but for any real usage, the package does not appear to have an issue here

@nate-strauser nate-strauser changed the title Entire cart history is publicly available Example app should not use insecure/autopublish Dec 8, 2014
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

2 participants