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

S3 Upload with CDN assumes path is persistent #3465

Open
xenithorb opened this issue Jun 5, 2016 · 1 comment
Open

S3 Upload with CDN assumes path is persistent #3465

xenithorb opened this issue Jun 5, 2016 · 1 comment

Comments

@xenithorb
Copy link
Contributor

xenithorb commented Jun 5, 2016

Your Rocket.Chat version: (make sure you are running the latest)
0.32.0

The documentation for the S3 Bucket file storage is lacking (ill work on that later), and as a result some debugging and general trial and error was necessary to figure out how to get everything to come together.

  1. What was very confusing during the debugging process was understanding that any file uploaded while a CDN url is present is "stuck" on that particular path. I was going back and fourth wondering why the web console wasn't showing a new 302 location. General problem here is that there doesn't seem to be a way to move previous files, they're stuck at whatever path they were originally uploaded under.

  2. Show some kind of HTML5 template for the input boxes, as otherwise it's confusing to understand what it wants. The description actually says "CDN domain for downloads",

    screenshot_20160605_151704

    but this isn't true, as the 302 redirect, if using a domain, just appends it to the current URI (missing a scheme) - it should probably at least default to //. This is the root problem, as I followed the instructions to put the domain and was stuck after testing a few files because of Refactored and reorganised entire css/less files and logic. #1.

  3. Warnings should be present and obvious to the admin that currently, there's no way to back out of a domain choice or storage choice without serious consequences. Tools to migrate data between services (S3, filesystem, GridFS) and domains are missing

@xenithorb
Copy link
Contributor Author

This should probably be re-titled "File uploading assumes path stays persistent" or "Rocket.Chat in need of data migration tool" but I'm going to leave that to a mod in case they want to keep it specific.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants