Skip to content

Commit 6247492

Browse files
fisxCopilot
andcommitted
Better effect action name.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
1 parent 3b9fd8b commit 6247492

4 files changed

Lines changed: 5 additions & 5 deletions

File tree

libs/wire-subsystems/src/Wire/AuthenticationSubsystem.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ data AuthenticationSubsystem m a where
7878
SameLabelPolicy ->
7979
AuthenticationSubsystem m (Either RetryAfter (Cookie (ZAuth.Token t)))
8080
RevokeCookies :: UserId -> [CookieId] -> [CookieLabel] -> AuthenticationSubsystem m ()
81-
RevokeAllExpiredCookies :: UserId -> AuthenticationSubsystem m ()
81+
RevokeAllStaleCookies :: UserId -> AuthenticationSubsystem m ()
8282
-- Verification Codes
8383
EnforceVerificationCodeEither :: Local UserId -> Maybe Code.Value -> VerificationAction -> AuthenticationSubsystem m (Either VerificationCodeError ())
8484
-- For testing

libs/wire-subsystems/src/Wire/AuthenticationSubsystem/Cookie.hs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -148,14 +148,14 @@ revokeCookiesMatchingExcept u mself ids labels = do
148148
-- Remove stale cookies. Stale means either (1) cookie is expired, or
149149
-- (2) cookie creation time is further in the past than
150150
-- `env.suspendInactiveUsers` allows.
151-
revokeAllExpiredCookiesImpl ::
151+
revokeAllStaleCookiesImpl ::
152152
( Member SessionStore r,
153153
Member (Input AuthenticationSubsystemConfig) r,
154154
Member Now r
155155
) =>
156156
UserId ->
157157
Sem r ()
158-
revokeAllExpiredCookiesImpl uid = do
158+
revokeAllStaleCookiesImpl uid = do
159159
now :: UTCTime <- Now.get
160160
mbSuspendAge <- (.suspendInactiveUsers) <$> input
161161

libs/wire-subsystems/src/Wire/AuthenticationSubsystem/Interpreter.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ interpretAuthenticationSubsystem userSubsystemInterpreter =
107107
NewCookie uid mcid typ mLabel policy -> newCookieImpl uid mcid typ mLabel policy
108108
NewCookieLimited uid mcid typ mLabel policy -> runError $ newCookieLimitedImpl uid mcid typ mLabel policy
109109
RevokeCookies uid ids labels -> revokeCookiesImpl uid ids labels
110-
RevokeAllExpiredCookies uid -> revokeAllExpiredCookiesImpl uid
110+
RevokeAllStaleCookies uid -> revokeAllStaleCookiesImpl uid
111111
-- Verification Codes
112112
EnforceVerificationCodeEither luid mCode action -> runError $ enforceVerificationCodeImpl luid mCode action
113113
-- Testing

services/brig/src/Brig/API/User.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -679,7 +679,7 @@ changeSingleAccountStatusInternal status ev u = do
679679
-- considered an account activity, so users that have their status
680680
-- changed recently should not be considered inactive, even if they
681681
-- haven't taken any action themselves.
682-
Auth.revokeAllExpiredCookies u
682+
Auth.revokeAllStaleCookies u
683683
UserStore.updateAccountStatus u status
684684
User.internalUpdateSearchIndex u
685685
Events.generateUserEvent u Nothing (ev u)

0 commit comments

Comments
 (0)