Skip to content

Commit

Permalink
[temp accounts] Set expiration to 90 days
Browse files Browse the repository at this point in the history
Why:

- We are unlikely to see good faith editing patterns with temp accounts
  that would require a full year

What:

- Set the default expiry for temp accounts to 90 days

Bug: T359653
Change-Id: Iae9dd0f73aceecfc9935b2b6019b035b1057eeb3
  • Loading branch information
kostajh committed Jun 20, 2024
1 parent 72580a3 commit 499277a
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions docs/config-schema.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4833,7 +4833,7 @@ config-schema:
reservedPattern: { type: [string, 'null'], default: ~$1 }
serialProvider: { type: object, default: { type: local, useYear: true } }
serialMapping: { type: object, default: { type: plain-numeric } }
expireAfterDays: { type: [integer, 'null'], default: 365 }
expireAfterDays: { type: [integer, 'null'], default: 90 }
notifyBeforeExpirationDays: { type: [integer, 'null'], default: 10 }
type: object
description: |-
Expand Down Expand Up @@ -4894,7 +4894,7 @@ config-schema:
- uppercase: (bool) With "filtered-radix", whether to use uppercase
letters, default false.
- offset: (int) With "plain-numeric", a constant to add to the stored index.
- expireAfterDays: (int|null, default 365) If not null, how many days should the temporary
- expireAfterDays: (int|null, default 90) If not null, how many days should the temporary
accounts expire? Requires expireTemporaryAccounts.php to be periodically executed in
order to work.
- notifyBeforeExpirationDays: (int|null, default 10) If not null, how many days before the
Expand Down
4 changes: 2 additions & 2 deletions includes/MainConfigSchema.php
Original file line number Diff line number Diff line change
Expand Up @@ -7671,7 +7671,7 @@ public static function getDefaultMetaNamespace( $sitename ): string {
* - uppercase: (bool) With "filtered-radix", whether to use uppercase
* letters, default false.
* - offset: (int) With "plain-numeric", a constant to add to the stored index.
* - expireAfterDays: (int|null, default 365) If not null, how many days should the temporary
* - expireAfterDays: (int|null, default 90) If not null, how many days should the temporary
* accounts expire? Requires expireTemporaryAccounts.php to be periodically executed in
* order to work.
* - notifyBeforeExpirationDays: (int|null, default 10) If not null, how many days before the
Expand All @@ -7690,7 +7690,7 @@ public static function getDefaultMetaNamespace( $sitename ): string {
'reservedPattern' => [ 'type' => 'string|null', 'default' => '~$1' ],
'serialProvider' => [ 'type' => 'object', 'default' => [ 'type' => 'local', 'useYear' => true ] ],
'serialMapping' => [ 'type' => 'object', 'default' => [ 'type' => 'plain-numeric' ] ],
'expireAfterDays' => [ 'type' => 'int|null', 'default' => 365 ],
'expireAfterDays' => [ 'type' => 'int|null', 'default' => 90 ],
'notifyBeforeExpirationDays' => [ 'type' => 'int|null', 'default' => 10 ],
],
'type' => 'object',
Expand Down
2 changes: 1 addition & 1 deletion includes/config-schema.php
Original file line number Diff line number Diff line change
Expand Up @@ -1156,7 +1156,7 @@
'serialMapping' => [
'type' => 'plain-numeric',
],
'expireAfterDays' => 365,
'expireAfterDays' => 90,
'notifyBeforeExpirationDays' => 10,
],
'AutoblockExpiry' => 86400,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ trait TempUserTestTrait {
*/
private function getTempAccountConfigTestDefaults(): array {
return [
'expireAfterDays' => 365,
'expireAfterDays' => 90,
'notifyBeforeExpirationDays' => 10,
'actions' => [ 'edit' ],
'genPattern' => '~$1',
Expand Down

0 comments on commit 499277a

Please sign in to comment.