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

Policies: export get_uri method on context #1245

Merged
merged 1 commit into from
Jan 12, 2021
Merged

Conversation

eloycoto
Copy link
Contributor

When using routing policy in APIAP, the path is always set to /. Becuase
routing policy is always the first, the jwt_claim_check started to fail,
because ngx.var.uri is always /.

This created a shared method, get_uri, and if routing policy was in use,
it'll use the new path.

FIX THREESCALE-6410

Signed-off-by: Eloy Coto [email protected]

@eloycoto eloycoto requested a review from a team as a code owner November 30, 2020 11:36
@eloycoto eloycoto force-pushed the URI_FIX branch 4 times, most recently from ef4cb92 to 0ce67c3 Compare November 30, 2020 15:23
When using routing policy in APIAP, the path is always set to /. Becuase
routing policy is always the first, the jwt_claim_check started to fail,
because ngx.var.uri is always /.

This created a shared method, get_uri, and if routing policy was in use,
it'll use the new path.

FIX THREESCALE-6410

Signed-off-by: Eloy Coto <[email protected]>
@eloycoto eloycoto merged commit 2048172 into 3scale:master Jan 12, 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.

2 participants