JOSESwift icon indicating copy to clipboard operation
JOSESwift copied to clipboard

[JWE] Added support for `A256GCM` and `A128GCM`

Open tobihagemann opened this issue 3 years ago • 9 comments

After considering the comments in #251, I've added support for content encryption using AES GCM according to JWE standard. As I've mentioned there, I increased the minimum iOS target to 13.0 because of CryptoKit.

tobihagemann avatar Aug 12 '22 15:08 tobihagemann

Thanks for this PR, any thoughts on completing it and releasing new lib version?

istarp avatar Nov 01 '22 21:11 istarp

From my perspective, it's completed. I'm just waiting for a review.

tobihagemann avatar Nov 02 '22 10:11 tobihagemann

I hope someone will take a look, seems like most of the PR's are not reviewed at all :(. It would be awesome to have this functionality as we are currently integrating Play Services Integrity check (Google version of device check from apple) and need this to decrypt token as it uses A256GCM

istarp avatar Nov 02 '22 14:11 istarp

Hello 👋🏽 Are any chances this PR will be merged?

antoniopantaleo avatar Jun 12 '23 08:06 antoniopantaleo

After considering the comments in #251, I've added support for content encryption using AES GCM according to JWE standard. As I've mentioned there, I increased the minimum iOS target to 13.0 because of CryptoKit.

Any update on this. Please include AES128GCM and AES256GCM content encryption algorithm. It would be very helpful

kiransable279 avatar Aug 25 '23 11:08 kiransable279

Hi, any update on this? It will be very useful to have this PR merged.

lsolniczek avatar Sep 25 '23 13:09 lsolniczek

Looks like it fails lint tests - @tobihagemann

Thireus avatar Oct 20 '23 09:10 Thireus

Fixed linter warning and brought branch up-to-date. But I'm not sure if these factors were actually blockers for a review. 😅

tobihagemann avatar Oct 24 '23 13:10 tobihagemann

Is there anything we can do to help get this PR merged in?

sfostermc avatar Jan 23 '24 20:01 sfostermc