-
Notifications
You must be signed in to change notification settings - Fork 1
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
Remove backend route and use Caddy to route requests #842
Comments
@paulushcgcj You're right, this is better. 👍 |
Is this still relevant @DerekRoberts and @mishraomp? What would be the benefit of doing this? I mean, the backend API will still be exposed if this is the concern. Another thing that can be problematic is the rate-limiting, so I was wondering about the relevance of this. We can discuss that to evaluate if and when this issue can become some task to be worked on. |
The main advantage of removing the route to removing the exposure of the entire API. |
@paulushcgcj It also prevents hard coding, which has been an issue on some projects. Like poiting their DEV PR instances at TEST or, worse still, PROD backends. |
@paulushcgcj another point, if we remove the exposure to internet, then no need of CORS, which avoids the OPTIONS API call, means lower latency :) |
may be a ticket in the backlog to align this with QSOS , thoughts?
Originally posted by @mishraomp in #829 (comment)
The text was updated successfully, but these errors were encountered: