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

feat: hide internal services behind proxy #281

Merged
merged 2 commits into from
Jan 30, 2024
Merged

feat: hide internal services behind proxy #281

merged 2 commits into from
Jan 30, 2024

Conversation

torives
Copy link
Contributor

@torives torives commented Jan 24, 2024

Modifies HttpService to act as a reverse proxy. Calls to /graphql, /inspect, /rollups, and /metrics endpoints are now received by it and forwarded to graphql-server, inspect-server, host-runner, and dispatcher respectively. The ports these services use needn't be exposed anymore, making HttpService the single access point to the node's APIs.

This is a step towards transforming these services into Golang modules.

Closes #241

@torives torives added no changelog PRs that don't require changes in changelog #feat:go-supervisor Feature: Go supervisor labels Jan 24, 2024
@torives torives self-assigned this Jan 24, 2024
@torives torives force-pushed the feature/proxy branch 2 times, most recently from 755f6f2 to 19344bd Compare January 24, 2024 17:24
Modifies HttpService to act as a reverse proxy. Calls to /graphql, /inspect, /rollups, and /metrics endpoints are now received by it and forwarded to graphql-server, inspect-server, host-runner, and dispatcher respectively. The ports used by these services needn't be exposed anymore, making HttpService the single point of access to the node's APIs.

This a is a step towards the goal of transforming these services into golang modules.
@gligneul gligneul merged commit d0f017a into main Jan 30, 2024
6 checks passed
@gligneul gligneul deleted the feature/proxy branch January 30, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#feat:go-supervisor Feature: Go supervisor no changelog PRs that don't require changes in changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setup HTTP proxy to internal endpoints
2 participants