AppAuth-Android icon indicating copy to clipboard operation
AppAuth-Android copied to clipboard

Change or skip allowed time skew for ID token issue time validation

Open Mardaneus86 opened this issue 2 years ago • 6 comments

Checklist

  • [x] I read the Contribution Guidelines
  • [x] I signed the CLA and WG Agreements
  • [x] I ran, updated and added unit tests as necessary.
  • [x] I verified the contribution matches existing coding style.
  • [x] I updated the documentation if necessary.

Motivation and Context

Change aims to address issue in #830 by adding the ability to either disable or change the allowed time skew for the ID Token issued at time (iat). Changing the allowed time skew can be useful because the clock on some devices can go beyond the default of 10 minutes. The OIDC specs do not define a specific timeframe, and the default of 10 minutes is still used when the new options are not used.

Description

I followed a similar approach as #662 already did for skipping the issuer https check.

AppAuthConfiguration appAuthConfig = new AppAuthConfiguration.Builder()
    .setAllowedIssueTimeSkew(THIRTY_MINUTES_IN_SECONDS)
    .build()
AppAuthConfiguration appAuthConfig = new AppAuthConfiguration.Builder()
    .setSkipIssueTimeValidation(true)
    .build()

Mardaneus86 avatar Jan 04 '24 09:01 Mardaneus86

This is a feature we've been waiting for for so long! Please review and accept, and we can move on from depending on v0.7.1 of the library :)

brighthr-stanton avatar Jan 16 '24 10:01 brighthr-stanton

Hi repo maintainers :) , Any news on this? Have been waiting for a month.... @WilliamDenniss @iainmcgin @StevenEWright

brighthr-stanton avatar Feb 27 '24 10:02 brighthr-stanton

agologan I see you seem to be one of the maintainers now? forgive me if I've misunderstood that. Any response to this PR please? We have been waiting a long time for this to be fixed.

brighthr-stanton avatar Mar 13 '24 17:03 brighthr-stanton

Any news on this ?

sanduluca avatar Apr 19 '24 10:04 sanduluca

@agologan Can you check this PR please ?

sanduluca avatar Jul 11 '24 13:07 sanduluca