8000 fix(coderd): ensure that clearing invalid oauth refresh tokens works with dbcrypt by johnstcn · Pull Request #15721 · coder/coder · GitHub
[go: up one dir, main page]
Skip to content

fix(coderd): ensure that clearing invalid oauth refresh tokens works with dbcrypt #15721

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

Merged
merged 4 commits into from
Dec 3, 2024
Merged

fix(coderd): ensure that clearing invalid oauth refresh tokens works with dbcrypt #15721

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
77954e1
fix(coderd): ensure that clearing invalid oauth refresh tokens works …
johnstcn Dec 2, 2024
e6cdd1f
DBMEEEEEEEEEEEEEEEMMMMMMMMMMMMMMMMMMMMMMM
johnstcn Dec 2, 2024
e2ca180
dbmem yet again you stab me in the heart
johnstcn Dec 2, 2024
18837ba
prefer to pass a sql.NullString to dbCrypt.encryptField
johnstcn Dec 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
dbmem yet again you stab me in the heart
  • Loading branch information
@johnstcn
johnstcn committed Dec 2, 2024
commit e2ca180c25d690f68086bb4adac1e55aeee72501
2 changes: 1 addition & 1 deletion coderd/database/dbmem/dbmem.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8790,7 +8790,7 @@ func (q *FakeQuerier) UpdateExternalAuthLinkRefreshToken(_ context.Context, arg
continue
}
gitAuthLink.UpdatedAt = arg.UpdatedAt
gitAuthLink.OAuthRefreshToken = ""
gitAuthLink.OAuthRefreshToken = arg.OAuthRefreshToken
q.externalAuthLinks[index] = gitAuthLink

return nil
Expand Down
Loading
0