Skip to content

Conversation

@dbcfd
Copy link
Contributor

@dbcfd dbcfd commented Oct 7, 2023

No description provided.

@dbcfd dbcfd requested a review from ukstv October 7, 2023 02:20
Copy link
Contributor

@ukstv ukstv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, conditional to return await changes. It is not a big deal. Just a little bit more sanity.

did: this._id,
...options,
payload,
return await createJWSUsing({
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You do not have to do return await here. And, createJWS could be unmarked async. Wrapping/unwrapping promises adds an overhead. Same for other return await clauses.

@dbcfd dbcfd force-pushed the feat/threads branch 2 times, most recently from 585033b to b0ab22f Compare January 29, 2024 15:19
Exposes necessary functionality to use threads for jws verification.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants