feat: add querystringStringifier to config

fastify · Sep 12, 2022 · 63eaa1a · 63eaa1a
1 parent 045059e
commit 63eaa1a
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 0 deletions.
diff --git a/README.md b/README.md
@@ -113,6 +113,21 @@ proxy.register(require('@fastify/reply-from'), {
 })
 ```
 
+#### `querystringStringifier`
+
+Set the default query-string stringifier
+[`fast-querystring`](https://npmjs.com/package/fast-querystring) will be used by default
+if you do not set a query-string stringifier.
+
+```js
+const querystring = require('querystring')
+
+proxy.register(require('@fastify/reply-from'), {
+  base: 'https://localhost:3001',
+  querystringStringifier: (str) => querystring.stringify(str),
+})
+```
+
 #### `http`
 
 Set the `http` option to an Object to use 

diff --git a/index.js b/index.js
@@ -22,6 +22,10 @@ const {
 } = require('./lib/errors')
 
 module.exports = fp(function from (fastify, opts, next) {
+  if (opts.querystringStringifier && typeof opts.querystringStringifier !== 'function') {
+    throw new Error(`querystringStringifier option should be a function, instead got '${typeof opts.querystringStringifier}'`)
+  }
+
   const contentTypesToEncode = new Set([
     'application/json',
     ...(opts.contentTypesToEncode || [])
@@ -201,6 +205,10 @@ function getQueryString (search, reqUrl, opts) {
   }
 
   if (opts.queryString) {
+    if (opts.querystringStringifier) {
+      return '?' + opts.querystringStringifier(opts.queryString)
+    }
+
     return '?' + querystring.stringify(opts.queryString)
   }
 

diff --git a/types/index.d.ts b/types/index.d.ts
@@ -91,6 +91,7 @@ export interface FastifyReplyFromOptions {
   retryMethods?: (HTTPMethods | 'TRACE')[];
   maxRetriesOn503?: number;
   disableRequestLogging?: boolean;
+  querystringStringifier?: (str: { [key: string]: unknown } ) => string
 }
 
 declare const fastifyReplyFrom: FastifyPluginCallback<FastifyReplyFromOptions>;