fix(typescript): b64: true is fine to use in JWT, its useless, but al…

…lowed (#324)
panva · Nov 18, 2021 · ee401c9 · ee401c9
1 parent 444cc3f
commit ee401c9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/jwt/sign.ts b/src/jwt/sign.ts
@@ -46,6 +46,7 @@ export class SignJWT extends ProduceJWT {
  if (
  Array.isArray(this._protectedHeader?.crit) &&
  this._protectedHeader.crit.includes('b64') &&
+ // @ts-expect-error
  this._protectedHeader.b64 === false
  ) {
  throw new JWTInvalid('JWTs MUST NOT use unencoded payload')

diff --git a/src/types.d.ts b/src/types.d.ts
@@ -734,7 +734,7 @@ export interface CompactJWSHeaderParameters extends JWSHeaderParameters {
  * may also be present.
  */
 export interface JWTHeaderParameters extends CompactJWSHeaderParameters {
- b64?: never
+ b64?: true
 }
 
 /**