Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Depends on #282
After merging #282, there would be no sense in this option. The
decodeUriParameters
option is used to optimize decoding non-Latin chars in path params. After #282 we won't parse params from the raw path but will parse them from the decoded path by the decodeURI function. The only chars that we will need to decode in params are# $ & + , / : ; = ? @ %
. We can do it natively in the find-my-may in the most optimized wayfind-my-way/lib/url-sanitizer.js
Line 69 in d834596
If you didn't add any side effects to this function, it doesn't change anything for you except increasing performance.
Fastify doesn't use the
decodeUriParameters
option.