fix: support type UUID
from node's crypto
package
#2797
Merged
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.
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Using the
UUID
type from node'scrypto
package with the@nestjs/swagger
plugin enabled crashes nestjs startnpm run start
with a cryptic error message.Example dto
This problem goes away when the
IsUUID()
decorator fromclass-validator
package is used. But considering the fact that UUID is an aliastype UUID = `${string}-${string}-${string}-${string}-${string}`;
the inferred type without any help should bestring
Issue Number: N/A
What is the new behavior?
After this change, if the plugin is enabled, and no decorator is used on a property of type
crypto.UUID
then its type gets inferred asstring
in the openapi spec.Does this PR introduce a breaking change?
Other information