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

Improve NGINX integration doc, clarify hostname/URL envvars #910

Closed
7 tasks
djwfyi opened this issue Jul 5, 2023 · 2 comments · Fixed by #921
Closed
7 tasks

Improve NGINX integration doc, clarify hostname/URL envvars #910

djwfyi opened this issue Jul 5, 2023 · 2 comments · Fixed by #921

Comments

@djwfyi
Copy link
Collaborator

djwfyi commented Jul 5, 2023

The NGINX integration doc needs some tweaks to clarify how to get things going.

Clarify the purpose and use of the following envvar and document:

  • MINIO_SERVER_URL
  • MINIO_BROWSER_REDIRECT_URL
  • MINIO_DOMAIN

Clarify the purpose and use of the following NGINX properties for MinIO

  • location
  • proxy_pass

Update the following pages as needed to reflect their use:

@djwfyi
Copy link
Collaborator Author

djwfyi commented Jul 5, 2023

See minio/console#2818 (comment)

@ravindk89 ravindk89 changed the title Improve NGINX integration doc Improve NGINX integration doc, clarify hostname/URL envvars Jul 5, 2023
@ravindk89
Copy link
Collaborator

Minor confirmation @harshavardhana

  • MINIO_SERVER_URL sets the FQDN MinIO uses for any operation that requires returning a hostname. This is important for MinIO Server under a reverse proxy to ensure MinIO uses that proxy address in responses vs an internal hostname or IP

  • MINIO_BROWSER_REDIRECT_URL sets the FQDN MinIO Console uses for any operation that requires returning a hostname. Same basic logic.

If you specify MINIO_SERVER_URL only, the Console may not respond to certain requests (e.g. presigned URL) using the correct hostname when behind a reverse proxy.

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 a pull request may close this issue.

2 participants