Skip to content

Commit be2ad57

Browse files
committed
Rename token hashed_secret to refresh_hash
1 parent cc366f6 commit be2ad57

File tree

8 files changed

+111
-111
lines changed

8 files changed

+111
-111
lines changed

coderd/database/dbgen/dbgen.go

+6-6
Original file line numberDiff line numberDiff line change
@@ -719,12 +719,12 @@ func OAuth2ProviderAppCode(t testing.TB, db database.Store, seed database.OAuth2
719719

720720
func OAuth2ProviderAppToken(t testing.TB, db database.Store, seed database.OAuth2ProviderAppToken) database.OAuth2ProviderAppToken {
721721
token, err := db.InsertOAuth2ProviderAppToken(genCtx, database.InsertOAuth2ProviderAppTokenParams{
722-
ID: takeFirst(seed.ID, uuid.New()),
723-
CreatedAt: takeFirst(seed.CreatedAt, dbtime.Now()),
724-
ExpiresAt: takeFirst(seed.CreatedAt, dbtime.Now()),
725-
HashedSecret: takeFirstSlice(seed.HashedSecret, []byte("hashed-secret")),
726-
AppSecretID: takeFirst(seed.AppSecretID, uuid.New()),
727-
APIKeyID: takeFirst(seed.APIKeyID, uuid.New().String()),
722+
ID: takeFirst(seed.ID, uuid.New()),
723+
CreatedAt: takeFirst(seed.CreatedAt, dbtime.Now()),
724+
ExpiresAt: takeFirst(seed.CreatedAt, dbtime.Now()),
725+
RefreshHash: takeFirstSlice(seed.RefreshHash, []byte("hashed-secret")),
726+
AppSecretID: takeFirst(seed.AppSecretID, uuid.New()),
727+
APIKeyID: takeFirst(seed.APIKeyID, uuid.New().String()),
728728
})
729729
require.NoError(t, err, "insert oauth2 app token")
730730
return token

coderd/database/dbmem/dbmem.go

+6-6
Original file line numberDiff line numberDiff line change
@@ -5372,12 +5372,12 @@ func (q *FakeQuerier) InsertOAuth2ProviderAppToken(_ context.Context, arg databa
53725372
if secret.ID == arg.AppSecretID {
53735373
//nolint:gosimple // Go wants database.OAuth2ProviderAppToken(arg), but we cannot be sure the structs will remain identical.
53745374
token := database.OAuth2ProviderAppToken{
5375-
ID: arg.ID,
5376-
CreatedAt: arg.CreatedAt,
5377-
ExpiresAt: arg.ExpiresAt,
5378-
HashedSecret: arg.HashedSecret,
5379-
APIKeyID: arg.APIKeyID,
5380-
AppSecretID: arg.AppSecretID,
5375+
ID: arg.ID,
5376+
CreatedAt: arg.CreatedAt,
5377+
ExpiresAt: arg.ExpiresAt,
5378+
RefreshHash: arg.RefreshHash,
5379+
APIKeyID: arg.APIKeyID,
5380+
AppSecretID: arg.AppSecretID,
53815381
}
53825382
q.oauth2ProviderAppTokens = append(q.oauth2ProviderAppTokens, token)
53835383
return token, nil

coderd/database/dump.sql

+3-3
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

coderd/database/migrations/000186_oauth2_provider_codes.up.sql

+3-3
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,14 @@ CREATE TABLE oauth2_provider_app_tokens (
1515
id uuid NOT NULL,
1616
created_at timestamp with time zone NOT NULL,
1717
expires_at timestamp with time zone NOT NULL,
18-
hashed_secret bytea NOT NULL,
18+
refresh_hash bytea NOT NULL,
1919
app_secret_id uuid NOT NULL REFERENCES oauth2_provider_app_secrets (id) ON DELETE CASCADE,
2020
api_key_id text NOT NULL REFERENCES api_keys (id) ON DELETE CASCADE,
2121
PRIMARY KEY (id),
22-
UNIQUE(app_secret_id, hashed_secret)
22+
UNIQUE(app_secret_id, refresh_hash)
2323
);
2424

25-
COMMENT ON TABLE oauth2_provider_app_tokens IS 'Refresh tokens both provide a way to refresh an access tokens (API keys) and a way to link API keys with the OAuth2 app and secret that generated them.';
25+
COMMENT ON COLUMN oauth2_provider_app_tokens.refresh_hash IS 'Refresh tokens provide a way to refresh an access token (API key). An expired API key can be refreshed if this token is not yet expired, meaning this expiry can outlive an API key.';
2626

2727
-- When we delete a token, delete the API key associated with it.
2828
CREATE FUNCTION delete_deleted_oauth2_provider_app_token_api_key() RETURNS trigger

coderd/database/models.go

+7-7
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

coderd/database/queries.sql.go

+10-10
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

coderd/database/queries/oauth2.sql

+1-1
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ INSERT INTO oauth2_provider_app_tokens (
9898
id,
9999
created_at,
100100
expires_at,
101-
hashed_secret,
101+
refresh_hash,
102102
app_secret_id,
103103
api_key_id
104104
) VALUES(

0 commit comments

Comments
 (0)