Repository navigation
Conversation
|
@mcollina I have addressed your comments and would appreciate a review. Thanks! |
…ntification # Conflicts: # types/types.test-d.ts
Regenerate lib/version.js for 11.1.6 and run the generator from the npm "version" lifecycle hook, so the release workflow's `npm version` bump updates it and stages it (git add -u) together with package.json.
|
@mcollina this is rebased on main after the switch to node-redis (#347): the example and the benchmark now pass On your review comment: the version comes from a generated Lint, the 149 unit tests at 100% coverage and the tstyche run all pass locally. Could you approve the CI run and take another look? Thanks! |
Description
This PR adds Redis driver identification support using the
CLIENT SETINFOcommand, which helps Redis server administrators identify connections from the@fastify/sessionlibrary.Changes
getDefaultClientInfoTag()function that returns a version-tagged identifier (e.g.,fastify-session_v11.1.1)clientInfoTaglib/version.jsinstead ofpackage.jsonat runtime (bundler-safe);scripts/generate-version.jsruns on the npmversionhook and onprepack, andtest/version.test.jsfails if the file drifts frompackage.jsonstoreoption (node-redis example; ioredis takes the same option)Benefits
Related
Checklist
npm run test && npm run benchmark --if-present