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

Account for / routes in RestAdapter.allRoutes #488

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

Account for / routes in RestAdapter.allRoutes #488

wants to merge 1 commit into from

Commits on Oct 13, 2022

  1. Account for / routes in RestAdapter.allRoutes 

    Previously, there was a bug in RestAdapter.allRoutes where the paths for
routes which were mounted at / would return as empty instead of the
expected / because of some trailing slash removal logic that was faulty.
This commit fixes this so that it accounts for cases where the path
itself is /, so the logic will leave those unchanged.

Fixes strongloop#487
    @getsnoopy
    getsnoopy committed Oct 13, 2022
    Configuration menu
    Copy the full SHA
    a13b9af View commit details
    Browse the repository at this point in the history