-
Notifications
You must be signed in to change notification settings - Fork 289
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
Make ShareLink URL point to Console instead to MinIO Server #3255
Comments
Steps Required:
|
To clarify - @harshavardhana @bexsoft based on internal conversations, would this remove the need for us to use I've seen two user reports from community about having issues when using that envvar, vs not using it at all and simply specifying the FQDN of console to |
No @ravindk89 we still need that to communicate with MinIO, this is just sending the share files to console instead of MinIO. |
Thanks - I thought we had, at one point, discussed behavior where Console would always talk to MinIO over localhost? Or is that the case, and the |
is it possible to generate this URL programmatically, example from the CLI or from the Kubernetes Operator ? |
you can use |
and can those link be permanent ? as in, never expire ? |
No.. it is as per s3 Max 7 days
https://docs.aws.amazon.com/AmazonS3/latest/userguide/using-presigned-url.html for non expiring links, you can use |
Currently whenever we want to share a file, the share url contains a default url which if for some reason is not reacheable by the browser it will fail to access it.
The proposal is to make Console handle the GET of the object instead of sending them to MinIO server.
That way it doesn't matter what the MinIO server DNS is, since you can already access Console.
Right now:
Expected Behavior
Share url uses console backend url and port
Use some other endpoint e.g.
https://play.min.io:9443/api/v1/get-file/cestest/file-s2.txt?X-Amz-Algorithm=A...
Current Behavior
Share url uses 127.0.0.1 url
Steps to Reproduce (for bugs)
2.Click on an object
The text was updated successfully, but these errors were encountered: