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

Issue with Path Params Support in reverse_proxy_plug #173

Open
YevhenKysilov opened this issue Jan 25, 2024 · 1 comment
Open

Issue with Path Params Support in reverse_proxy_plug #173

YevhenKysilov opened this issue Jan 25, 2024 · 1 comment

Comments

@YevhenKysilov
Copy link

YevhenKysilov commented Jan 25, 2024

I am very excited about your project and was looking forward to integrating reverse_proxy_plug into my work project. During the integration process, I noticed that the plug seems to lack support for path parameters in the upstream URL. Specifically, path variables are not replaced with their corresponding values, leading to incorrect request routing.

Code Example:
forward "/users/:userId", to: ReverseProxyPlug, upstream: "https://example.com/users/:userId"

In this example, I expected :userId to be replaced by the actual user ID from the request, but this substitution does not occur.

Here is the solution to this issue
#174

@mwhitworth
Copy link
Collaborator

This is a good idea. I'll take a look and incorporate your changes.

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

No branches or pull requests

2 participants