[pull] main from fastify:main #52
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![pull[bot]](https://avatars.githubusercontent.com/in/12910?s=60&v=4){kind=small}
Signed-off-by: Matteo Collina <[email protected]>
# Conflicts: # fastify.js # package.json
# Conflicts: # fastify.js # package.json
# Conflicts: # fastify.js # package.json
# Conflicts: # fastify.js # package.json
Signed-off-by: RafaelGSS <[email protected]>
Signed-off-by: KaKa <[email protected]> # Conflicts: # docs/Reference/Errors.md # docs/Reference/Server.md # fastify.js # lib/route.js # package.json # test/default-route.test.js # test/types/instance.test-d.ts # test/types/route.test-d.ts
Signed-off-by: KaKa <[email protected]> # Conflicts: # fastify.js # package.json
* Disallow `decorate('name', null)` in the types
Preferred to use `decorate('name')` , `decorateRequest('name')`, `decorateReply('name')` variants instead
Deprecation not feasible because of microsoft/TypeScript#54872
Follow up to #4874
* Docs: Drop `null` from placeholder decorations
Signed-off-by: Pelle Wessman <[email protected]>
---------
Signed-off-by: Pelle Wessman <[email protected]>
Co-authored-by: Carlos Fuentes <[email protected]>
* add option to disable/ignore request-id header fixes #4192 * add more restrictive schema definition * update configValidator generated by running 'node build/build-validation.js' after change on 'build-validation.js' * move logic into reqIdGenFactory * change default value from requestIdHeader Changes default from 'request-id' to `false`. Making it opt-in instead of opt-out. NOTE: when user sets requestIdHeader to `true` the behavior is the same as if it was set to `false` because `req.headers[true] || genReqId(req)` is equivalent to `genReqId(req)` (req.headers[true] is undefined). TL;DR requestIdHeader must be a string, else it will be 'ignored'. Solution: Provide a default value for requestIdHeader when `true` or throw. * remove comment * adapt --------- Co-authored-by: Uzlopak <[email protected]>
* make FastifySchemaValidationError.params wider (#4476) * fix: FastifySchemaValidationError.params could be unknown * refactor: use FastifySchemaValidationError instead of ValidationResult * fix: export ValidationResult for backword compatibility * test: add non-assignable test for AjvErrorObject * docs(ecosystem): add fastify-hashids (#4934) * fix: hasPlugin does not track parent plugins (#4929) Co-authored-by: Carlos Fuentes <[email protected]> * docs: early hints plugin is fastify plugin (#4947) * docs: early hints plugin is fastify plugin * fix * Update docs/Guides/Ecosystem.md Co-authored-by: Frazer Smith <[email protected]> --------- Co-authored-by: Frazer Smith <[email protected]> * chore: add pull request title check (#4951) * chore: add pull request title check * Update .github/workflows/pull-request-title.yml Co-authored-by: Frazer Smith <[email protected]> --------- Co-authored-by: Frazer Smith <[email protected]> * chore: Bump @sinclair/typebox from 0.29.6 to 0.30.2 (#4952) Bumps [@sinclair/typebox](https://github.com/sinclairzx81/typebox) from 0.29.6 to 0.30.2. - [Commits](sinclairzx81/typebox@0.29.6...0.30.2) --- updated-dependencies: - dependency-name: "@sinclair/typebox" dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * ci: prove pr title check (#4953) * improve pr title check * Apply suggestions from code review * fix * fix * Update .github/workflows/pull-request-title.yml Co-authored-by: Luis Orbaiceta <[email protected]> * Apply suggestions from code review --------- Co-authored-by: Luis Orbaiceta <[email protected]> * ci: fix warnings in benchmark workflows (#4954) * improve pr title check * Apply suggestions from code review * fix * fix * ci: fix warnings in benchmark workflow * Update .github/workflows/benchmark.yml * Update .github/workflows/benchmark.yml * docs: fix removeAdditional comment (#4948) * docs(Validator Compiler): fix removeAdditional comment * Update docs/Reference/Validation-and-Serialization.md Co-authored-by: Uzlopak <[email protected]> * fix: typo in comment * Update docs/Reference/Validation-and-Serialization.md Co-authored-by: Frazer Smith <[email protected]> --------- Co-authored-by: Uzlopak <[email protected]> Co-authored-by: Frazer Smith <[email protected]> * fix: Try to fix parser benchmark workflow (#4956) * fix: infer correct hook handler (#4945) * fix: infer correct hook handler * Update test/types/hooks.test-d.ts * simplify * remove duplicate typing tests * remove duplicate test * remove unused import * fix * fix linting * fix: remove http version check * test: remove unsupported http version * remove obsolete documentation --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: cm-ayf <[email protected]> Co-authored-by: Anderson <[email protected]> Co-authored-by: Manuel Spigolon <[email protected]> Co-authored-by: Carlos Fuentes <[email protected]> Co-authored-by: Uzlopak <[email protected]> Co-authored-by: Frazer Smith <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Luis Orbaiceta <[email protected]> Co-authored-by: Mohammed Gomaa <[email protected]> Co-authored-by: Igor Savin <[email protected]> Co-authored-by: Cem <[email protected]>
Co-authored-by: Carlos Fuentes <[email protected]>
# Conflicts: # fastify.js # lib/warnings.js # package.json # test/default-route.test.js # test/listen.deprecated.test.js # test/serial/logger.0.test.js # test/serial/logger.1.test.js # test/stream.test.js
# Conflicts: # docs/Reference/Errors.md # fastify.js # lib/pluginUtils.js # lib/route.js # lib/server.js # lib/warnings.js # package.json # test/default-route.test.js # test/plugin.4.test.js # types/instance.d.ts
# Conflicts: # lib/configValidator.js
* feat: change `reply.redirect()` signature * feat: change `reply.redirect()` signature * docs * docs * update message * fix deprecation * update message
* feat: emit diagnostics_channel upon handling request * use TracingChannel#hasSubscribers * update dc-polyfill to fix a hang in c8 lib
Bumps [pnpm/action-setup](https://github.com/pnpm/action-setup) from 3 to 4. - [Release notes](https://github.com/pnpm/action-setup/releases) - [Commits](pnpm/action-setup@v3...v4) --- updated-dependencies: - dependency-name: pnpm/action-setup dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
in decorate.js
* docs(ecosystem): Add fastify-kysely plugin * fix alphabetical order
* chore: update tap@18 * chore: tap@19 * chore: remove c8 * chore: remove ignore
Signed-off-by: codershiba <[email protected]>
* chore: remove unused (and failing) test * add version check to test * Update test/close-pipelining.test.js Co-authored-by: KaKa <[email protected]> Signed-off-by: Cangit <[email protected]> * lint: remove semver pkg --------- Signed-off-by: Cangit <[email protected]> Co-authored-by: KaKa <[email protected]>
* type: readyListener can be async on server.ready() * fix lint warning var -> const
* migrate to neostandard Signed-off-by: Matteo Collina <[email protected]> * fixup Signed-off-by: Matteo Collina <[email protected]> * Update eslint.config.js Co-authored-by: Pelle Wessman <[email protected]> Signed-off-by: Matteo Collina <[email protected]> * fixup Signed-off-by: Matteo Collina <[email protected]> * fixup Signed-off-by: Matteo Collina <[email protected]> * fixup Signed-off-by: Matteo Collina <[email protected]> --------- Signed-off-by: Matteo Collina <[email protected]> Signed-off-by: Matteo Collina <[email protected]> Co-authored-by: Pelle Wessman <[email protected]>
* fix:hasRoute method comparison with case insensitive * fix:rename variable inside hasRoute method and update test case * fix normalizedMethod with optional chaining and nullish coalescing
* feat(types): Introduce SafePromiseLike Signed-off-by: Roberto Bianchi <[email protected]> * Update doc Signed-off-by: Roberto Bianchi <[email protected]> * Update brand property Signed-off-by: Roberto Bianchi <[email protected]> * Update types/type-provider.d.ts Co-authored-by: Josh Goldberg ✨ <[email protected]> Signed-off-by: Roberto Bianchi <[email protected]> * Update docs/Reference/TypeScript.md Co-authored-by: Josh Goldberg ✨ <[email protected]> Signed-off-by: Roberto Bianchi <[email protected]> * Fix link Signed-off-by: Roberto Bianchi <[email protected]> * Update docs/Reference/TypeScript.md Co-authored-by: Josh Goldberg ✨ <[email protected]> Signed-off-by: Roberto Bianchi <[email protected]> * Add safePromiseLike tests Signed-off-by: Roberto Bianchi <[email protected]> * Add line break Signed-off-by: Roberto Bianchi <[email protected]> * fix lint Signed-off-by: Roberto Bianchi <[email protected]> * Update docs/Reference/TypeScript.md Co-authored-by: Josh Goldberg ✨ <[email protected]> Signed-off-by: Roberto Bianchi <[email protected]> * Update docs/Reference/TypeScript.md Co-authored-by: Josh Goldberg ✨ <[email protected]> Signed-off-by: Roberto Bianchi <[email protected]> --------- Signed-off-by: Roberto Bianchi <[email protected]> Signed-off-by: Roberto Bianchi <[email protected]> Co-authored-by: Josh Goldberg ✨ <[email protected]>
* docs: addContentTypeParser with fastify.register Signed-off-by: moradebianchetti81 <[email protected]> * fix: add example doc contentTypeParse * implement suggested changes * Update docs/Reference/ContentTypeParser.md Signed-off-by: Frazer Smith <[email protected]> --------- Signed-off-by: moradebianchetti81 <[email protected]> Signed-off-by: Frazer Smith <[email protected]> Co-authored-by: Frazer Smith <[email protected]>
Co-authored-by: hanquliu <[email protected]>
Signed-off-by: Matteo Collina <[email protected]>
Bumps the dependencies-major group with 2 updates: [@fastify/error](https://github.com/fastify/fastify-error) and [fast-json-stringify](https://github.com/fastify/fast-json-stringify). Updates `@fastify/error` from 3.4.1 to 4.0.0 - [Release notes](https://github.com/fastify/fastify-error/releases) - [Commits](fastify/fastify-error@v3.4.1...v4.0.0) Updates `fast-json-stringify` from 5.16.1 to 6.0.0 - [Release notes](https://github.com/fastify/fast-json-stringify/releases) - [Commits](fastify/fast-json-stringify@v5.16.1...v6.0.0) --- updated-dependencies: - dependency-name: "@fastify/error" dependency-type: direct:production update-type: version-update:semver-major dependency-group: dependencies-major - dependency-name: fast-json-stringify dependency-type: direct:production update-type: version-update:semver-major dependency-group: dependencies-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps the dev-dependencies group with 1 update: [neostandard](https://github.com/neostandard/neostandard). Updates `neostandard` from 0.7.2 to 0.8.0 - [Release notes](https://github.com/neostandard/neostandard/releases) - [Changelog](https://github.com/neostandard/neostandard/blob/main/CHANGELOG.md) - [Commits](neostandard/neostandard@v0.7.2...v0.8.0) --- updated-dependencies: - dependency-name: neostandard dependency-type: direct:development update-type: version-update:semver-minor dependency-group: dev-dependencies ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )