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

[policy][dynamic-routing-policy] allow to not encode path #2239

Closed
NicolasGeraud opened this issue May 14, 2019 · 0 comments

Comments

@NicolasGeraud
Copy link
Member

commented May 14, 2019

Some apis used encoded values in as path params :

If you use the dynamic-routing policy, this encoded values is decoded in som cases.

We have to provide a way to use the raw url

Steps to Reproduce (for bugs)

  1. add the policy
  2. fill the target with https://www.googleapis.com/storage/v1/b/example-bucket/o/foo%2fbar
  3. call your api
  4. see in logs that https://www.googleapis.com/storage/v1/b/example-bucket/o/foo/bar has been called by gravitee

Your Environment

  • Version used: 1.25.x

@NicolasGeraud NicolasGeraud added this to the APIM - 1.25.x milestone May 14, 2019

phiz71 added a commit to gravitee-io/gravitee-gateway-api that referenced this issue May 15, 2019

fix(): add a new attribute in ExecutionContext so the dynamic-routing…
… policy can send context information to the gateway

Fix gravitee-io/issues#2239

phiz71 added a commit to gravitee-io/gravitee-policy-dynamic-routing that referenced this issue May 15, 2019

phiz71 added a commit to gravitee-io/gravitee-gateway that referenced this issue May 15, 2019

fix(): use execution context to check wether the resolver should retu…
…rn the raw uri or the encoded uri (default behavior)

Fix gravitee-io/issues#2239

NicolasGeraud added a commit to gravitee-io/gravitee-gateway that referenced this issue May 21, 2019

fix(): use execution context to check wether the resolver should retu…
…rn the raw uri or the encoded uri (default behavior)

Fix gravitee-io/issues#2239

NicolasGeraud added a commit to gravitee-io/gravitee-gateway that referenced this issue May 21, 2019

fix(): use execution context to check wether the resolver should retu…
…rn the raw uri or the encoded uri (default behavior)

Fix gravitee-io/issues#2239

NicolasGeraud added a commit to gravitee-io/gravitee-gateway-api that referenced this issue May 21, 2019

fix(): add a new attribute in ExecutionContext so the dynamic-routing…
… policy can send context information to the gateway

Fix gravitee-io/issues#2239

NicolasGeraud added a commit to gravitee-io/gravitee-policy-dynamic-routing that referenced this issue May 21, 2019

@aelamrani aelamrani reopened this May 28, 2019

@gravitee-io gravitee-io deleted a comment from aelamrani May 29, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.