Proposal for variable routes

[anddave]

Proposal for variable routes

• As of RestPS 7.0.44 from PSGallery, only hardcoded routes are supported, which is to say that Invoke-RequestRouter selects routes where {$_.RequestURL -eq $RequestURL}.
• Consequently endpoints like "api/periodictable/{element}" are marginally feasible in small scale only and not feasible in large scale.
• Proposal: we could add a regex match to Invoke-RequestRouter which would enable variable routes.
• For the sake of simplicity, I propose that:
  • Map file should only be changed to support regex matching
  • as opposed to adding any more notational load to the map file, let the endpoint scripts do all the interpreting of variable routes beyond matching the regex.
• Some defensive code and unit testing may need to be added for mitigating the risk of accidental route doubling if a route is both an exact match and a regex match.
• Branch and PR forthcoming soon Thanks @jpsider great module

[jpsider]

Makes sense. I have not had the need to add this up to this point. But in dealing with other API's use this type of thing on a normal basis. I am looking forward to reviewing the PR!

[anddave]

This is what it looks like when multiple routes match

INFO: Start-RestPSListener: Processing RequestType: GET URL: /periodictable Args: /RestPS/RestPS/private/Invoke-RequestRouter.ps1:58 Line | 58 | $CommandReturn = . $RequestCommand -RequestArgs $RequestA … | ~~~~~~~~~~~~~~~ | The term | '/periodictable/endpoints/GET/Invoke-EndpointPeriodicTable.ps1 | /periodictable/endpoints/GET/Invoke-EndpointPeriodicTable.ps1' | is not recognized as a name of a cmdlet, function, script | file, or executable program. Check the spelling of the name, | or if a path was included, verify that the path is correct and | try again.