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.

Sign up for GitHub

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

Jump to bottom

Use nullish coalescing to determine path #661

Open
wants to merge 1 commit into
base: mainline
Choose a base branch
from
Open

Use nullish coalescing to determine path #661

wants to merge 1 commit into from

Commits on Oct 31, 2023

  1. Use nullish coalescing to determine path 

    We are currently struggling with a situation where `/test` proxies to a lambda. This causes `event.pathParameters.proxy` to be `undefined`. The request is then translated to `/test` rather than `/`.

To solve this we want to overwrite `event.pathParameters.proxy` with an empty string but the absence of nullish coalescing causes this to still use the `event.path` rather than `event.pathParameters.proxy`
    @sanderkoenders
    sanderkoenders authored Oct 31, 2023
    Configuration menu
    Copy the full SHA
    6008d16 View commit details
    Browse the repository at this point in the history