Every captured commit receives deterministic security triage and a separate communication-quality score. Security candidates and broader second-pass signals receive full-patch Ollama analysis.
Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.
This commit only edits the wording of a changelog entry for BTCPay Server 2.4.2. It does not change any code, configuration, or executable files. The edit makes the release sound more urgent, but the commit itself is purely documentation a…
Changelog language changed to describe a critical, actively exploited vulnerability in release 2.4.2No code, configuration, or dependency changes present in the commitSecurity issue is referenced but not fixed or explained in this diff
This commit marks several public methods on ASP.NET Core controllers with [NonAction], which prevents them from being exposed as web-accessible HTTP routes. Without this attribute, these helper methods could be reached directly via URL, po…
Public controller methods exposed as unintended HTTP routesRouting-level access-control hardeningPotential bypass of intended controller action flow
This commit is a version bump and changelog for BTCPay Server 2.4.2. The changelog describes two security-related changes: disabling Basic authentication by default shortly after account creation, and fixing a TOTP two-factor authenticatio…
Vendor describes security fixes in changelogTOTP two-factor authentication bypass via Greenfield Basic authenticationBasic authentication disabled by default after account creation
This commit only adds, removes, and reorders English text strings used for translations and user-interface labels in BTCPay Server. There are no code logic changes, no security settings being enabled or disabled, and no behavior changes. I…
This commit updates several third-party software libraries and a test Docker image to newer versions. It also removes one library (AngleSharp) that was no longer needed. Dependency updates can fix security bugs in those external packages, …
Dependency version bumps for packages that handle HTML sanitization (HtmlSanitizer) and Bitcoin protocol parsing (NBitcoin/NBXplorer)Removal of unused AngleSharp package referenceNo commit-level description of security fixes or CVEs
This change makes BTCPay Server's Greenfield API stop accepting username-and-password 'Basic' authentication by default for existing users. New accounts can still use it for only the first five minutes after creation to set up an API key. …
Disables Basic authentication by default after a 5-minute onboarding windowAdds per-user opt-in flag for Basic auth in data model, API, and UITightens rate-limit bypass to require a recently created account
This commit is a user-interface redesign, not a security fix. It wraps store settings pages in new visual 'section' containers, updates headings and spacing, and adjusts CSS styling. No code behavior, permissions, or data handling changed.
BTCPay Server fixed a bug where accounts protected only by TOTP-based two-factor authentication (2FA) could access the Greenfield API using just an email and password, skipping the second factor. The change now blocks Basic authentication …
This commit adds a user-facing preference feature to BTCPay Server's wallet transactions table: a dropdown that lets users show, hide, and reorder columns. Preferences are saved in the browser's localStorage. The change is a routine UI enh…
No security-relevant keywords in commit title or messagePure client-side UI feature using localStorageNo changes to authentication, authorization, or input processing
This commit changes how BTCPay Server decides whether a Bitcoin transaction input is a SegWit (witness) type when building PSBTs (Partially Signed Bitcoin Transactions). Previously the code only recognized three specific SegWit formats: na…
Broadens SegWit detection from a fixed allow-list to any ScriptType.Witness, mitigating future-version compatibility issuesAdds missing WitnessUtxo population in two wallet PSBT creation pathsWitness UTXO data reduces reliance on NonWitnessUtxo (full previous transaction) for SegWit inputs, which is a known PSBT hardening recommendation
This commit removes a single line that automatically added witness UTXO data to SegWit inputs when decoding a PSBT. The change is titled 'Preserve imported PSBTs,' suggesting the previous behavior was modifying user-supplied transaction da…
PSBT mutation behavior changedWitness UTXO data no longer auto-injected for SegWit inputsPotential data-integrity / non-repudiation consideration for imported PSBTs
This commit fixes a bookkeeping bug in BTCPay Server's multisig (multi-signature) transaction tracker. Previously, a pending transaction could silently collect enough signatures to be finalized, but the system would still label it as 'Pend…
State-machine inconsistency: persisted transaction state could diverge from actual signature countMultisig workflow: incorrect 'Pending' label for a transaction that is actually ready to broadcastNo input validation or authorization changes in the diff
This commit changes BTCPay Server so that when it builds or decodes Partially Signed Bitcoin Transactions (PSBTs), it explicitly adds a compact 'witness UTXO' record for any SegWit inputs. Previously, some SegWit PSBTs may only have carrie…
PSBT data completeness change for SegWit inputsPotential prior state: SegWit PSBTs may have been distributed without witness UTXOsHardware-wallet compatibility / BIP-174 compliance improvement
This commit is a follow-up code review patch for BTCPay Server's pending Bitcoin transaction (multisig) signing feature. It tightens when the service logs a warning versus a debug message after a failed finalization attempt, and it removes…
Logging level change reduces false-warning noise but could mask real finalization failures if SignaturesNeeded is incorrectly zeroRemoval of additionalPsbt parameter prevents accidental combination of an unvalidated extra PSBT into the effective PSBTPer-input signature progress tracking refined to avoid prematurely counting aggregate signatures
This commit is a minor code cleanup that simplifies how BTCPay Server writes log messages when a pending multi-signature transaction fails to finalize. It removes the list of failed input indexes from the log and merges two nearly identica…
This change tightens how BTCPay Server stores signature attempts for multi-signature Bitcoin transactions. Previously, every submitted PSBT (a partially signed transaction file) was kept in history, even if it added no real progress. Now o…
Unbounded history/list growth bounded by logical retention gateResource consumption / storage abuse vector mitigatedLogic change in multi-signature signature collection path
This commit only adds more detailed logging when a pending multi-signature Bitcoin transaction fails to finalize. It does not change how transactions are validated, authorized, or executed. There is no security vulnerability being fixed he…
This commit fixes a bug in BTCPay Server's multisig transaction handling. Previously, the service could get stuck in a 'Pending' state even when enough valid signatures had been collected, because it discarded new signature submissions tha…
Logic flaw in multisig signature collection could leave transactions permanently pending despite sufficient valid signaturesPreviously ignored distinct PSBTs that did not immediately increase signature progressMissing retry of transaction finalization after collecting additional signatures
This commit changes one word in the project's changelog, adding the word 'the' to a bullet point. It is a grammatical typo fix with no effect on software behavior, security, or user data.
This commit changes how BTCPay Server stores invoice comments and metadata. Previously, comments were saved in a separate field and could be updated with a method that was not safe when multiple requests happened at the same time. The patc…
Commit title explicitly describes a race condition fixReplaced read-modify-write metadata update with atomic PostgreSQL jsonb_set operationRemoved dedicated API 'comment' field, narrowing update surface
Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.
Lower-priorityAdd plugin compatibility regression check to release CI (#7394)by Chukwuleta Tobechi · e2472a5f · Jun 9, 2026 · 2 filesMessage 83 · StrongTriage 0Details
Commit message · Chukwuleta Tobechi
Add plugin compatibility regression check to release CI (#7394)
* Add plugin compatibility regression check to release CI
* resolve code rabbit
* clean
83/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
Lower-priorityChangelog for 2.4.0 (#7393)by Nicolas Dorier · b0742a55 · Jun 9, 2026 · 3 filesMessage 53 · ThinTriage 0Details
Commit message · Nicolas Dorier
Changelog for 2.4.0 (#7393)
53/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
AI review queuedUpdate translationsby Nicolas Dorier · 85c8c379 · Jun 8, 2026 · 1 fileMessage 18 · OpaqueInformational 15Details
Commit message · Nicolas Dorier
Update translations
18/100 · OpaqueMessage clarity
✓ Subject identifies a change! Too few words to establish purpose! No meaningful explanatory body
This commit only updates translation strings in a single C# resource file. It adds, removes, and rewords user-facing text labels (for example, changing '2FA' wording to 'Authenticator', adding labels for new passkey and multisig wallet features, and removing unused strings). There are no code logic changes, no security fixes, and no behavior changes.
AI review queuedFix flaky testby Nicolas Dorier · b705089d · Jun 8, 2026 · 1 fileMessage 38 · OpaqueInformational 15Details
Commit message · Nicolas Dorier
Fix flaky test
38/100 · OpaqueMessage clarity
✓ Subject identifies a change✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100
This commit adds a single line to a test file to wait for the web page to finish loading before checking its content. It is a test reliability fix, not a security fix, and does not change any production code.
AI review queuedfix buildby Nicolas Dorier · dfb66912 · Jun 8, 2026 · 1 fileMessage 0 · OpaqueInformational 15Details
Commit message · Nicolas Dorier
fix build
0/100 · OpaqueMessage clarity
! Very short subject! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100
This is a one-line change in a test file that updates the name of a database migration being tested. It fixes a broken build caused by a renamed migration and has no security relevance.
Lower-priorityCheck plugin compatibility as part of the Pre-Release checksby Nicolas Dorier · 3d9bb12c · Jun 8, 2026 · 2 filesMessage 50 · ThinTriage 0Details
Commit message · Nicolas Dorier
Check plugin compatibility as part of the Pre-Release checks
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: unusually broad changesecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit adds a new multisignature wallet setup feature to BTCPay Server. It introduces new database tables, UI flows, controllers, services, and tests for creating and managing multisig wallets. The changes are a feature implementation, not a security fix or vulnerability patch. There is no indication in the commit or supplied references that this addresses a security issue.
AI review queuedAvoid spurious crash GetCachedStoreDataby Nicolas Dorier · e9379494 · Jun 4, 2026 · 1 fileMessage 35 · OpaqueLow 35Details
Commit message · Nicolas Dorier
Avoid spurious crash GetCachedStoreData
35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
This commit swaps a regular dictionary for a thread-safe concurrent dictionary when caching store data attached to a web request. The stated goal is to prevent rare, random server crashes caused by multiple threads accessing the cache at the same time. It is a hardening fix rather than a clear-cut exploitable vulnerability, and the commit message does not frame it as a security issue.
Security candidateShow missing permission in 403 page (#7387)by Nicolas Dorier · 09493a99 · Jun 3, 2026 · 4 filesMessage 53 · ThinInformational 15Details
Commit message · Nicolas Dorier
Show missing permission in 403 page (#7387)
53/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
Why it was queued
credential or privilege state
AI analysis · Informational 15/100
This change is purely a user-experience improvement: when a BTCPay Server user is denied access to a page, the 403 error page now tells them which specific permission they are missing. It does not alter who can access what, nor does it fix or introduce any security vulnerability. It simply makes the denial message more informative.
Lower-priorityBetter handle malformed walletId in the storeDataby Nicolas Dorier · 7a3e5f97 · Jun 3, 2026 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · Nicolas Dorier
Better handle malformed walletId in the storeData
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
subscription: allows credit refunds via Pull Payments (#7284)
58/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
Lower-priorityFix: Archiving invoices with a custom range filter threw error 403 (#7386)by Nicolas Dorier · 54a138bb · Jun 2, 2026 · 5 filesMessage 70 · AdequateTriage 0Details
Commit message · Nicolas Dorier
Fix: Archiving invoices with a custom range filter threw error 403 (#7386)
70/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
Lower-priorityallow for notification email in subscription (#7300)by Chukwuleta Tobechi · 8dae67e4 · Jun 2, 2026 · 6 filesMessage 83 · StrongTriage 0Details
Commit message · Chukwuleta Tobechi
allow for notification email in subscription (#7300)
* allow for notification email in subscription
* include test
83/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
Security candidateInclude configuration for tax on tips (#7298)by Chukwuleta Tobechi · 5a025a84 · Jun 2, 2026 · 16 filesMessage 76 · AdequateInformational 19Details
Commit message · Chukwuleta Tobechi
Include configuration for tax on tips (#7298)
* Include configuration for tax on tips
* Ensure null and 0 result in no tip tax
* mirroring tax in tip scenario in CanUsePOSKeypad.
* Format tip tax receipt
* Update
* fix coderabbit suggestion
* Include tip tax to report, and remove tax from product information
76/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Why it was queued
boot or update path
AI analysis · Informational 19/100
This commit adds a new optional 'tax on tips' feature to BTCPay Server's Point of Sale app. Merchants can now configure a separate tax rate that applies to customer tips, and the receipt/cart display breaks out the tip tax separately from regular item tax. There is no indication this is a security fix; it is a normal business-logic feature addition.
Lower-priorityDo not allow submission of auth code if it is not activatedby Nicolas Dorier · a819bc58 · Jun 1, 2026 · 1 fileMessage 55 · ThinTriage 0Details
Commit message · Nicolas Dorier
Do not allow submission of auth code if it is not activated
55/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Names security-relevant behavior explicitly! No meaningful explanatory body
Security candidateRemove recovery codesby Nicolas Dorier · 9010e3bd · Jun 1, 2026 · 14 filesMessage 28 · OpaqueLow 35Details
Commit message · Nicolas Dorier
Remove recovery codes
28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body! Opaque security-relevant change
Why it was queued
authentication path
AI analysis · Low 35/100
This commit removes the 2FA recovery-code feature from BTCPay Server. Recovery codes are normally printed when you enable two-factor authentication and used as a backup way to log in if you lose your phone. After this change, users can no longer create or use those codes. The commit does not add a replacement backup login path, so anyone who loses access to their authenticator app may be permanently locked out of their account unless they have another login method configured. The change is presented as a feature removal, not as a fix for a specific security bug.
Security candidateAdd loginless and passwordless passkey authenticationby Lucas Cullen · 28604914 · Jun 1, 2026 · 52 filesMessage 55 · ThinModerate 59Details
Commit message · Lucas Cullen
Add loginless and passwordless passkey authentication
55/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Names security-relevant behavior explicitly! No meaningful explanatory body
Why it was queued
access controlauthentication path
AI analysis · Moderate 59/100
This commit adds passkey (passwordless) and login-code login support to BTCPay Server and rewrites much of the existing two-factor/FIDO2/LNURL-auth login flow. It is a large feature patch, not a documented security fix. The changes introduce several security-relevant design choices: passkeys can bypass the password entirely, session state now carries the login method and return URL, and the migration forces TwoFactorEnabled=true for all users while adding a separate AuthenticatorEnabled flag. Because the patch is broad and partially refactored, there is a non-trivial risk of authentication bugs (e.g., bypasses, session confusion, or incorrect 2FA enforcement), but the supplied diff does not show an obvious exploitable vulnerability.
AI review queuedBump to xUnit v3 (#7382)by napoly · 43fc81c8 · May 30, 2026 · 34 filesMessage 46 · ThinInformational 15Details
Commit message · napoly
Bump to xUnit v3 (#7382)
46/100 · ThinMessage clarity
✓ Subject identifies a change✓ Names a concrete action or component✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: unusually broad changesecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit updates the project's test framework from xUnit version 2 to xUnit version 3. It only changes test code, package references, and compiler warning suppressions. There is no change to the production application that users interact with, and no security issue is introduced or fixed.
The keypad POS view (VueLight.cshtml) was missing the `.posItem-added` overlay element that Cart.cshtml has. `common.js` uses a `transitionend` listener on `.posItem` elements to remove the `posItem--added` class after the overlay's opacity fade completes. Without the overlay there is no transition, so `transitionend` never fires on iOS Safari, leaving `pointer-events: none` on the row permanently after the first tap.
On Android an incidental Bootstrap button transition happened to fire `transitionend` and accidentally clear the class, masking the bug.
Add the missing overlay to VueLight.cshtml so the transition fires consistently on all platforms.
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
81/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
* test(wallet): remove reflection from filter fast test
100/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 22/100
This commit adds new search and filter features to the wallet transaction list in BTCPay Server. Users can now filter transactions by text, direction (incoming/outgoing), labels, and date ranges, and export results matching those filters. The change is a feature addition with a large amount of new code, but the diff itself does not show an obvious security vulnerability. There are a few minor hardening touches, such as adding a missing null check for an unknown cryptocurrency code and removing a redundant anti-forgery attribute.
Adds a new public rate provider operated by bitcoin.co.ke (Kenyan Bitcoin community service) covering BTC against KES and 40+ other African and major currencies. Provides redundancy for Kenyan BTCPay merchants whose only reliable KES rate source today is Bitnob.
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
AI review queuedUpdate docsby Nicolas Dorier · 6eb9649a · May 19, 2026 · 1 fileMessage 0 · OpaqueInformational 15Details
Commit message · Nicolas Dorier
Update docs
0/100 · OpaqueMessage clarity
! Very short subject! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100
This commit only updates the API documentation file (a Swagger/OpenAPI template) to add a description for a newly documented setting called 'storeQuota'. It does not change any server code, permissions, or behavior. There is no security issue in this change.
Security candidatefeat: allow max stores per user (#7320)by Abhijay Jain · e6708c47 · May 19, 2026 · 14 filesMessage 80 · StrongInformational 22Details
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Links an issue, advisory, or supporting reference
Why it was queued
boot or update path
AI analysis · Informational 22/100
This commit adds a new feature that lets server administrators set a maximum number of stores each non-admin user can create, both globally and per-user. It is a new restriction/control feature, not a fix for an existing vulnerability. There is no evidence in the commit or supplied references that this change addresses a security incident or was disclosed as security-relevant.
Security candidateUnnest UI views of PullRequests, PullPayments, Invoices and Apps (#7368)by Nicolas Dorier · c7c90625 · May 19, 2026 · 24 filesMessage 58 · ThinLow 38Details
Commit message · Nicolas Dorier
Unnest UI views of PullRequests, PullPayments, Invoices and Apps (#7368)
58/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
Why it was queued
authentication path
AI analysis · Low 38/100
This commit restructures several user-interface pages so that payment requests, pull payments, invoices and apps no longer live under a '/stores/{storeId}/...' URL path. Instead they use flatter routes such as '/payment-requests/{id}/edit'. The change also updates the authorization layer so that when a store-scoped route is missing but an object ID is present, the system can fail the request with a 403 rather than letting the action itself silently return a 404. Several tests were updated to expect 403 instead of 404 for missing objects, and some permission-guard tests were removed or changed. The commit is described by the author as a UI route cleanup, not as a security fix, but it does touch access-control code.