Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

EZP-29574: the time life of the token has been adjusted to the same as in the platform #3135

Merged
merged 1 commit into from
Jan 19, 2022
Merged

EZP-29574: the time life of the token has been adjusted to the same as in the platform #3135

merged 1 commit into from
Jan 19, 2022

Conversation

mateuszdebinski
Copy link
Contributor

@mateuszdebinski mateuszdebinski commented Jan 18, 2022 •
edited

Question Answer
JIRA issue EZP-29574
Type bug/improvement
Target eZ Platform version v2.5
BC breaks no

As there are still sometimes problems with tests and not finding the user when updating the token, the token lifetime has been changed according to the time it is on the platform

ezsettings.default.security.token_interval_spec: PT1H

Checklist:

  • Provided PR description.
  • Tested the solution manually.
  • Provided automated test coverage.
  • Checked that target branch is set correctly (master for features, the oldest supported for bugs).
  • Ran PHP CS Fixer for new PHP code (use $ composer fix-cs).
  • Asked for a review (ping @ezsystems/engineering-team).

@mateuszdebinski mateuszdebinski self-assigned this Jan 18, 2022
@mateuszdebinski mateuszdebinski requested a review from a team January 18, 2022 15:40
@konradoboza konradoboza requested a review from a team January 19, 2022 07:09
@ViniTou ViniTou requested a review from a team January 19, 2022 07:33
@Steveb-p Steveb-p requested a review from a team January 19, 2022 08:27
@adamwojs adamwojs merged commit ccab771 into 7.5 Jan 19, 2022
@adamwojs adamwojs deleted the EZP-29574-integration-test-randomly-fails branch January 19, 2022 08:27
@mateuszdebinski mateuszdebinski mentioned this pull request Jan 19, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Steveb-p Steveb-p Steveb-p approved these changes

@ViniTou ViniTou ViniTou approved these changes

@konradoboza konradoboza konradoboza approved these changes

Assignees

@mateuszdebinski mateuszdebinski

Labels
Bug Ready for review
Milestone
No milestone
5 participants
@mateuszdebinski @Steveb-p @ViniTou @konradoboza @adamwojs