@Deactivated User thanks for opening that PR to update jwks-rsa. Can you also take a look at Issue #434 (and PR 435, that’s a proposed fix)?
d
Deactivated User
06/07/2018, 3:05 PM
Np. Yeah I have seen it, and looks fine to me, but other person, probably Sergey should validate and merge it.
I would also add a test verifying the behaviour if possible
m
mkporwit
06/07/2018, 3:06 PM
OK, I’ll try and add a test case for that.
👍 1
mkporwit
06/07/2018, 5:38 PM
@Deactivated User OK, done. Added a test case and switched to using the