auth0 / express-jwt

connect/express middleware that validates a JsonWebToken (JWT) and set the req.user with the attributes
MIT License
4.49k stars 444 forks source link

fix: corrected files in package.json #327

Closed krichards closed 1 year ago

krichards commented 1 year ago

Description

Fixed files in package.json so that pnpm installs correctly. See https://github.com/auth0/express-jwt/issues/326

Testing

Create a project with pnpm

mkdir test cd test pnpm init pnpm i express-jwt

Check the contents of node_modules

ls node_modules/express-jwt LICENSE package.json README.md

The dist folder will be missing

Build express jwt from the fix

npm run build npm pack

This should generate a file express-jwt-8.4.1.tgz. Update the package json file

... "express-jwt": "file:/....path-to-file/express-jwt-8.4.1.tgz" ...

pnpm i

Check the installed express-jwt folder under node_modules and you should see

ls node_modules/express-jwt dist LICENSE package.json README.md

Checklist

krichards commented 1 year ago

Looks like it was a red herring - the pnpm install works correctly on a fresh monorepo. It appeared to be a bad cached state.