bug: jti not used during JWT.sign

[alvis]

When using JWT.sign, jti can be supplied as an option for it to be included in the payload.
However, jti is currently not included in the payload generation
see below:

jose/lib/jwt/sign.js

Lines 65 to 74 in 10a1b64

	payload = {
	...payload,
	sub: subject || payload.sub,
	aud: audience || payload.aud,
	iss: issuer || payload.iss,
	iat: iat ? unix : payload.iat,
	nonce: nonce || payload.nonce,
	exp: expiresIn ? unix + secs(expiresIn) : payload.exp,
	nbf: notBefore ? unix + secs(notBefore) : payload.nbf
	}

Additional context
Add any other context about the problem here.

• the bug is happening on latest @panva/jose too.
• i have searched the issues tracker on github for similar issues and couldn't find anything related.

[panva]

@alvis thank you, i can see the issue now. (after having a second look) definitely something to fix to behave as documented.

[alvis]

Thank you @panva ;)

[panva]

@alvis v1.4.1