Per http://tools.ietf.org/html/rfc6749#section-5.1 the field expires_in in 'only' recommended in the access token returned by the openid connect server. Specifically coreos/dex doesn't return it.
Currently there is a warning because the filed is used without checking first if it exists.

CommentFileSizeAuthor
noexpire.patch1 KByaslama
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yaslama created an issue. See original summary.

pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins commented
Status: Active » Needs review

Seems right to me.

  • sanduhrs committed 8de674f on 8.x-1.x authored by yaslama 
    Issue #2642734 by yaslama: Support no expires_in field in the...
sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs at erdfisch commented
Status: Needs review » Reviewed & tested by the community

Looking good to me.
Commited and pushed to 8.x-1.x branch.

  • pjcdawkins committed 82ba5ed on 7.x-1.x authored by yaslama 
    Issue #2642734 by yaslama: Support no expires_in field in the...
pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x-1.x. Thanks

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.