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

Upstream: Do not append a trailing / on the end. #1298

Merged
merged 1 commit into from
Aug 17, 2021

Conversation

eloycoto
Copy link
Contributor

If the tmp_path is empty on upstream:append_path, we should add the
trailing slash, as resty_url:join is doing.

Fix https://issues.redhat.com/browse/THREESCALE-7146
Reported-by: sillumin@redhat.com

Signed-off-by: Eloy Coto eloy.coto@acalustra.com

@eloycoto eloycoto requested a review from a team as a code owner August 16, 2021 13:43
@eloycoto eloycoto force-pushed the THREESCALE-7146 branch 2 times, most recently from f9e4340 to f489bb9 Compare August 16, 2021 14:40
If the tmp_path is empty on upstream:append_path, we should add the
trailing slash, as resty_url:join is doing.

Fix https://issues.redhat.com/browse/THREESCALE-7146
Reported-by: sillumin@redhat.com

Signed-off-by: Eloy Coto <eloy.coto@acalustra.com>
@eloycoto eloycoto merged commit 0d43adf into 3scale:master Aug 17, 2021
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.

None yet

3 participants