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

Take into account the new path field. #31

Merged
merged 1 commit into from
Jun 22, 2017
Merged

Conversation

fredoche
Copy link
Contributor

@fredoche fredoche commented May 5, 2017

Hi,

ApiGateway provides an additional field. I believe it is related to x-ray but I can't be sure. It would be nice to expose it though, so lambda code can take advantage of it.

Hi,

ApiGateway provides an additional field. I believe it is related to x-ray but I can't be sure. It would be nice to expose it though, so lambda code can take advantage of it.
@sapessi
Copy link
Collaborator

sapessi commented May 5, 2017

I can't find this anywhere in the documentation, could you paste here a sample event with this field?

@fredoche
Copy link
Contributor Author

fredoche commented May 9, 2017

Hi,

Here's a real world example

{
"body": null,
"resource": "/version",
"requestContext": {
"resourceId": "xxx",
"apiId": "xxx",
"resourcePath": "/version",
"httpMethod": "GET",
"requestId": "xxx",
"accountId": "xxx",
"identity": {
"apiKey": "",
"userArn": null,
"cognitoAuthenticationType": null,
"caller": null,
"userAgent": "Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:53.0) Gecko/20100101 Firefox/53.0",
"user": null,
"cognitoIdentityPoolId": null,
"cognitoIdentityId": null,
"cognitoAuthenticationProvider": null,
"sourceIp": "xxx",
"accountId": null,
"accessKey": null
},
"authorizer": null,
"stage": "dev",
"path": "/dev/version"
},
"queryStringParameters": null,
"headers": {
"Accept": "application/json",
"Accept-Encoding": "gzip, deflate, br",
"Accept-Language": "en-US,en;q=0.5",
"CloudFront-Forwarded-Proto": "https",
"CloudFront-Is-Desktop-Viewer": "true",
"CloudFront-Is-Mobile-Viewer": "false",
"CloudFront-Is-SmartTV-Viewer": "false",
"CloudFront-Is-Tablet-Viewer": "false",
"CloudFront-Viewer-Country": "FR",
"DNT": "1",
"Host": "xxx",
"Origin": "xxx",
"Referer": "xxx",
"User-Agent": "Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:53.0) Gecko/20100101 Firefox/53.0",
"Via": "1.1 xxx.cloudfront.net (CloudFront)",
"X-Amz-Cf-Id": "xxx",
"X-Amzn-Trace-Id": "xxx",
"X-Forwarded-For": "xxx",
"X-Forwarded-Port": "443",
"X-Forwarded-Proto": "https"
},
"pathParameters": null,
"httpMethod": "GET",
"stageVariables": null,
"path": "/dev/version",
"base64Encoded": false
}

@sapessi
Copy link
Collaborator

sapessi commented May 9, 2017

Thanks, I spoke with the team and I was told to hold off on adding support for this. As soon as the API is stable we will add the field. I'll keep you posted here.

@sapessi sapessi merged commit c29d283 into aws:master Jun 22, 2017
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

2 participants