[PR #4077] [MERGED] fix: constant time compare #5172

Closed
opened 2026-03-13 12:12:54 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/better-auth/better-auth/pull/4077
Author: @himself65
Created: 8/19/2025
Status: Merged
Merged: 8/19/2025
Merged by: @himself65

Base: canaryHead: himself65/2025/08/18/compare


📝 Commits (2)

  • 48883b9 fix: constant time compare
  • a4082d9 fix: constant time compare

📊 Changes

1 file changed (+6 additions, -6 deletions)

View changed files

📝 packages/better-auth/src/crypto/buffer.ts (+6 -6)

📄 Description

An early return value will still lead to a timing attack issue

Related: https://github.com/better-auth/utils/pull/12


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/better-auth/better-auth/pull/4077 **Author:** [@himself65](https://github.com/himself65) **Created:** 8/19/2025 **Status:** ✅ Merged **Merged:** 8/19/2025 **Merged by:** [@himself65](https://github.com/himself65) **Base:** `canary` ← **Head:** `himself65/2025/08/18/compare` --- ### 📝 Commits (2) - [`48883b9`](https://github.com/better-auth/better-auth/commit/48883b9279da95c1c85721c2b8e3212a9864fb8e) fix: constant time compare - [`a4082d9`](https://github.com/better-auth/better-auth/commit/a4082d9603186bc8a3385e8c3641c181f85b9d07) fix: constant time compare ### 📊 Changes **1 file changed** (+6 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `packages/better-auth/src/crypto/buffer.ts` (+6 -6) </details> ### 📄 Description An early return value will still lead to a timing attack issue Related: https://github.com/better-auth/utils/pull/12 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-03-13 12:12:54 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/better-auth#5172