Skip to content

Conversation

@ChristophWurst
Copy link
Member

@ChristophWurst ChristophWurst commented May 7, 2024

Summary

  • Kill auth tokens that have not been used in a year
  • Kill auth/session tokens marked for remote wipe more than two months ago

Checklist

@ChristophWurst ChristophWurst self-assigned this May 7, 2024
@solracsf solracsf added this to the Nextcloud 30 milestone Jun 18, 2024
This was referenced Jul 30, 2024
@AndyScherzinger AndyScherzinger force-pushed the feat/auth/authtoken-clean-up branch from f15263a to ab9831a Compare August 1, 2024 13:41
@Altahrim Altahrim mentioned this pull request Aug 5, 2024
@ChristophWurst ChristophWurst force-pushed the feat/auth/authtoken-clean-up branch from ab9831a to cdf8f35 Compare August 5, 2024 16:23
@skjnldsv
Copy link
Member

skjnldsv commented Aug 6, 2024

Relevant

1) Test\Authentication\Token\PublicKeyTokenProviderTest::testInvalidateOldTokens
TypeError: Mock_IConfig_90b08448::getSystemValueInt(): Return value must be of type int, null returned

/home/runner/work/server/server/lib/private/Authentication/Token/PublicKeyTokenProvider.php:290
/home/runner/work/server/server/tests/lib/Authentication/Token/PublicKeyTokenProviderTest.php:354

@skjnldsv skjnldsv added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Aug 6, 2024
@Altahrim Altahrim mentioned this pull request Aug 7, 2024
@skjnldsv skjnldsv mentioned this pull request Aug 13, 2024
@ChristophWurst ChristophWurst force-pushed the feat/auth/authtoken-clean-up branch from cdf8f35 to 5100e31 Compare August 13, 2024 10:39
@ChristophWurst ChristophWurst added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Aug 13, 2024
@ChristophWurst ChristophWurst merged commit 8bf2095 into master Aug 13, 2024
@ChristophWurst ChristophWurst deleted the feat/auth/authtoken-clean-up branch August 13, 2024 14:32
@ChristophWurst ChristophWurst added the pending documentation This pull request needs an associated documentation update label Aug 19, 2024
@ChristophWurst
Copy link
Member Author

Documentation: nextcloud/documentation#12114

@ChristophWurst ChristophWurst removed the pending documentation This pull request needs an associated documentation update label Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants