Skip to content

[DNM] haproxy: add haproxy to rgw #96

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

nizamial09
Copy link
Member

@nizamial09 nizamial09 commented May 15, 2025

also configure cors by default

dc up -d haproxy would bring up the proxy and rgw will be serving in localhost:90 with cors enabled.

.env.example Outdated
@@ -33,6 +33,7 @@ KEYCLOAK_HOST_HTTPS_PORT=8443
#KEYCLOAK_IMAGE=
HAPROXY_HOST_HTTP_PORT=80
HAPROXY_HOST_HTTPS_PORT=443
HAPROXY_RGW_HOST_HTTP_PORT=90
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets use a range of custom ports rather than 90

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A custom port number is any port number outside of the standard or well-known ports (0-1023).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is that? When providing it to the docker-container to expose that port I think we can only give them a single port not the range isn't it?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated it to 9080

@nizamial09 nizamial09 force-pushed the haproxy-rgw-cors branch 2 times, most recently from cf5d2db to 74c5e13 Compare May 15, 2025 07:54
@nizamial09 nizamial09 changed the title haproxy: add haproxy to rgw [DNM] haproxy: add haproxy to rgw May 15, 2025
also configure cors by default

Signed-off-by: Nizamudeen A <[email protected]>
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 this pull request may close these issues.

2 participants