Skip to content

feat: hash tokens - #2

Merged
cstamas merged 4 commits into
trusted-publishing-tokensfrom
trusted-publishing-tokens-hash
Aug 7, 2026
Merged

feat: hash tokens#2
cstamas merged 4 commits into
trusted-publishing-tokensfrom
trusted-publishing-tokens-hash

Conversation

@cstamas

@cstamas cstamas commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Hash token values, also PAT table got version column that is 0 for existing tokens, and current version is 1.
"On the fly" upgrade added but also a job.

Changes:

  • token config got hashAlg + salt
  • token table got version smallint field (initialized to 0 on migration)
  • new job upgrades token (replaces value with hash(value + salt) and sets version = 1)
  • use token upgrade on the fly if (needed)
  • tests reworked to use less mocks

cstamas added 4 commits August 6, 2026 19:48
Hash token values, also PAT table got version column that is
0 for existing tokens, and current version is 1.
"On the fly" upgrade added but also a job.
@cstamas
cstamas merged commit fd01d78 into trusted-publishing-tokens Aug 7, 2026
@cstamas
cstamas deleted the trusted-publishing-tokens-hash branch August 7, 2026 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant