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

Headers policy: add more variables to liquid #1079

Closed
eloycoto opened this issue Jun 28, 2019 · 0 comments · Fixed by #1081
Closed

Headers policy: add more variables to liquid #1079

eloycoto opened this issue Jun 28, 2019 · 0 comments · Fixed by #1081
Assignees

Comments

@eloycoto
Copy link
Contributor

Discussed with a user,

Headers liquid policy does not have the request schema (http|htts) or the request port.

local function context_values()
return {
uri = ngx.var.uri,
host = ngx.var.host,
remote_addr = ngx.var.remote_addr,
headers = ngx.req.get_headers(),
http_method = ngx.req.get_method(),
}

@eloycoto eloycoto self-assigned this Jun 28, 2019
eloycoto added a commit to eloycoto/APIcast that referenced this issue Jun 28, 2019
Due to a user request, providing more variables to the Liquid templating
to allow headers policy (As axample) to get remote porrt, server address
and port and the scheme.

Fix 3scale#1079

Signed-off-by: Eloy Coto <eloy.coto@gmail.com>
eloycoto added a commit to eloycoto/APIcast that referenced this issue Jun 28, 2019
Due to a user request, providing more variables to the Liquid templating
to allow headers policy (As axample) to get remote porrt, server address
and port and the scheme.

Fix 3scale#1079

Signed-off-by: Eloy Coto <eloy.coto@gmail.com>
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 a pull request may close this issue.

1 participant