Skip to content
This repository has been archived by the owner on Jan 8, 2023. It is now read-only.

Extended options to allow custom path normalisation #56

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

pitrew
Copy link

@pitrew pitrew commented Mar 29, 2021

Extend options with custom path normalisation function

@pitrew pitrew force-pushed the extend-options-with-custom-path branch from 5218633 to c4f21e9 Compare March 29, 2021 14:13
@@ -73,6 +76,11 @@ app.use(promMid({
// // eslint-disable-next-line no-param-reassign
// labels.contentType = req.headers['content-type'];
// },
// customPathNormalizer(path) {
// if (path.includes('test')) {
// reutrn 'test';
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// reutrn 'test';
// return 'test';

@nlopin
Copy link

nlopin commented Jan 10, 2022

@joao-fontenele do you think you can merge this one, url-value-parser is pretty limited and it would be great to have an escape hatch

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants