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.

Sign up for GitHub

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

Jump to bottom

Storage::get_remote_files returns non-asset files #18

Closed
kranzky opened this issue Nov 9, 2011 · 2 comments
Closed

Storage::get_remote_files returns non-asset files #18

kranzky opened this issue Nov 9, 2011 · 2 comments

Comments

@kranzky
Copy link

kranzky commented Nov 9, 2011

Our bucket contains static assets uploaded by asset_sync (in an "assets" directory) as well as a bunch of other files (in other directories, such as "uploads"). Storage::get_remote_files returns everything. This is potentially dangerous, as Storage::delete_extra_remote_files would delete these non-asset files if we enabled it, and if issue 16 wasn't stopping Storage::get_remote_files from working.
@kranzky
Copy link
Author

kranzky commented Nov 9, 2011

The fix here is to specify :prefix => 'assets' when instantiating the bucket.

davidjrice added a commit that referenced this issue Nov 14, 2011
@davidjrice
Merge pull request #20 from agworld/e26f5ca36dee1c2196653268ed6bb38c0…
b941315 
…226e4d2

Fixes issues #16, #17, #18 and #19
@davidjrice
Copy link
Contributor

Thanks! Version 0.1.10 should solve this. Let me know if there are any further issues.

@ghost ghost mentioned this issue Oct 20, 2014
Open
jasukkas pushed a commit to AdsOptimal/asset_sync that referenced this issue Sep 15, 2016
fixes AssetSync/asset_sync#18
649dd67
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@davidjrice @kranzky