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

Set full URL path to request header #1266

Merged
merged 6 commits into from Aug 20, 2019

Conversation

@life1347
Copy link
Member

commented Aug 12, 2019

Resolve #1153

$ fission route list
NAME                                 METHOD HOST URL                INGRESS FUNCTION_NAME
40d95a72-8bc1-4c58-8b24-875ed942c410 GET         /foo/py1/{test} false   py1

# request headers
X-Fission-Function-Name:[py1]
X-Fission-Function-Namespace:[default]
X-Fission-Function-Resourceversion:[6443255]
X-Fission-Function-Uid:[376e1c54-bc15-11e9-82af-42010a8c0002]
X-Fission-Params-Test:[bar]
X-Fission-Full-Url:[/foo/py1/bar]

This change is Reviewable

@life1347 life1347 requested a review from soamvasani Aug 12, 2019

@life1347 life1347 force-pushed the request-full-url branch from 067389a to b8a6e5a Aug 12, 2019

@life1347 life1347 force-pushed the request-full-url branch from 8969de5 to 37cdae0 Aug 15, 2019

@vishal-biyani vishal-biyani added this to the 1.5 milestone Aug 15, 2019

life1347 added 2 commits Aug 19, 2019

@life1347 life1347 merged commit 8cafe1e into master Aug 20, 2019

6 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
security/snyk - environments/jvm/pom.xml (soamvasani) No new issues
Details
security/snyk - environments/nodejs/package.json (soamvasani) No manifest changes detected
security/snyk - environments/python/requirements.txt (soamvasani) No manifest changes detected
security/snyk - environments/ruby/Gemfile.lock (soamvasani) No manifest changes detected
security/snyk - examples/nodejs/package.json (soamvasani) No manifest changes detected

@life1347 life1347 deleted the request-full-url branch Aug 20, 2019

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