Skip to content

Commit

Permalink
Merge pull request #1963 from Panzer1119/fix-swagger-ui-init-js-gener…
Browse files Browse the repository at this point in the history
…ation

fix(@nestjs/swagger): replace quoted functionPlaceholder completely
  • Loading branch information
kamilmysliwiec committed Jul 11, 2022
2 parents 7ee0601 + 5c38773 commit 408e39d
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 1 deletion.
10 changes: 10 additions & 0 deletions e2e/manual-e2e.ts
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,16 @@ async function bootstrap() {
}
});

SwaggerModule.setup('/swagger-docs', app, document, {
customSiteTitle: 'Demo API - Swagger UI 3',
swaggerOptions: {
persistAuthorization: true,
defaultModelsExpandDepth: -1,
tagsSorter: (a, b) => a.localeCompare(b),
operationsSorter: (a, b) => a.get('id').localeCompare(b.get('id'))
}
});

USE_FASTIFY
? (app as NestFastifyApplication).useStaticAssets({
root: publicFolderPath,
Expand Down
2 changes: 1 addition & 1 deletion lib/swagger-ui/helpers.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ export function buildJSInitOptions(initOptions: SwaggerUIInitOptions) {
2
);

json = json.replace(new RegExp(functionPlaceholder, 'g'), () => fns.shift());
json = json.replace(new RegExp('"' + functionPlaceholder + '"', 'g'), () => fns.shift());

return `let options = ${json};`;
}

0 comments on commit 408e39d

Please sign in to comment.