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.
Security candidateAdd LUNO exchange as default for ZAR currencyby Nicolas Dorier · d27793dc · Jan 23, 2026 · 9 filesMessage 45 · ThinInformational 17Details
Commit message · Nicolas Dorier
Add LUNO exchange as default for ZAR currency
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
cryptography-sensitive path
AI analysis · Informational 17/100
This commit mainly adds the LUNO exchange as the default source for South African Rand (ZAR) exchange rates in BTCPay Server. It also includes a small hardening fix in another rate provider so that missing bid/ask values don't crash the app, plus a batch of unrelated test reliability improvements. There is no clear security vulnerability being patched.
Lower-priorityFix: Public node info page should not list local network ipsby Nicolas Dorier · 50d81267 · Jan 22, 2026 · 2 filesMessage 62 · AdequateTriage 0Details
Commit message · Nicolas Dorier
Fix: Public node info page should not list local network ips
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Lower-priorityTests: Rewrite CheckoutTests to make it more reliableby Nicolas Dorier · 342bd7b9 · Jan 22, 2026 · 4 filesMessage 60 · AdequateTriage 0Details
Commit message · Nicolas Dorier
Tests: Rewrite CheckoutTests to make it more reliable
60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
AI review queuedrefactor: Remove Selenium tests and testing infrastructureby Abhijay007 · b3d1a0a2 · Jan 21, 2026 · 18 filesMessage 87 · StrongInformational 15Details
Commit message · Abhijay007
refactor: Remove Selenium tests and testing infrastructure
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
87/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Mentions testing or verification
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit removes the old Selenium browser-automation test suite and its supporting infrastructure from the BTCPay Server project. It deletes test files, package references, Docker services, and CI job definitions, and updates two comments in production view files that previously mentioned Selenium. There is no change to the actual application code that users interact with, and no security fix or vulnerability is introduced.
Lower-priorityrefactor: migrated tests back in checkoutUI fileby Abhijay007 · d07466ef · Jan 21, 2026 · 2 filesMessage 82 · StrongTriage 0Details
Commit message · Abhijay007
refactor: migrated tests back in checkoutUI file
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
82/100 · StrongMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Mentions testing or verification
Lower-priorityrefactor: replaces contains to expectby Abhijay007 · f845f2f7 · Jan 21, 2026 · 1 fileMessage 72 · AdequateTriage 0Details
Commit message · Abhijay007
refactor: replaces contains to expect
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
Lower-priority(Refactor) : Converted Selenium test for CheckoutUITests to playwrightby Abhijay007 · 098b42b5 · Jan 21, 2026 · 3 filesMessage 75 · AdequateTriage 0Details
Commit message · Abhijay007
(Refactor) : Converted Selenium test for CheckoutUITests to playwright
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
75/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
Lower-priorityTests: Fix subscriptions flaky testsby Nicolas Dorier · 53e06c7e · Jan 21, 2026 · 2 filesMessage 55 · ThinTriage 0Details
Commit message · Nicolas Dorier
Tests: Fix subscriptions flaky tests
55/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
AI review queuedTests: Move tests to PullPaymentTests.csby Nicolas Dorier · 834fab4f · Jan 21, 2026 · 3 filesMessage 55 · ThinInformational 15Details
Commit message · Nicolas Dorier
Tests: Move tests to PullPaymentTests.cs
55/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit is purely a test-code reorganization. It moves several Playwright UI tests from one test file to another and creates a new dedicated WalletTests.cs file. No production application code was changed, so it cannot introduce a security vulnerability in the software users run.
AI review queuedTests: Move WebhooksTests in the same classby Nicolas Dorier · f545a9b6 · Jan 21, 2026 · 6 filesMessage 55 · ThinInformational 15Details
Commit message · Nicolas Dorier
Tests: Move WebhooksTests in the same class
55/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100
This commit is a test-code refactoring: it moves existing webhook-related tests from several scattered test files into a single new file called WebhooksTests.cs. No production code is changed, and the commit message explicitly says it is just moving tests. There is no security fix or vulnerability here.
AI review queuedMake tests more reliableby Nicolas Dorier · 46e06266 · Jan 21, 2026 · 1 fileMessage 38 · OpaqueInformational 15Details
Commit message · Nicolas Dorier
Make tests more reliable
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 only changes test code in BTCPay Server's PlaywrightTests.cs. It refactors how browser tabs are switched during automated UI tests and replaces some polling-style assertions with more direct Playwright expectations. There is no change to production code, no security fix, and no user-facing behavior change.
Lower-priorityRefactor: Move some leftover boltcards from PullPayment to pluginsby Nicolas Dorier · 0d4a0236 · Jan 21, 2026 · 4 filesMessage 62 · AdequateTriage 0Details
Commit message · Nicolas Dorier
Refactor: Move some leftover boltcards from PullPayment to plugins
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Security candidateFix flaky CanCreateCrowdfundingAppby Nicolas Dorier · 69cc179f · Jan 21, 2026 · 2 filesMessage 35 · OpaqueInformational 15Details
Commit message · Nicolas Dorier
Fix flaky CanCreateCrowdfundingApp
35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body! Opaque security-relevant change
Why it was queued
boot or update path
AI analysis · Informational 15/100
This commit fixes a flaky automated test for the crowdfunding feature. The test previously cleared a date field by directly manipulating the webpage's internal value, which was unreliable. The change makes the test click a visible 'Clear' button instead, and adds an ID to that button so the test can find it. There is no security issue here.
Lower-priorityRefactor: Move boltcards related code to pluginsby Nicolas Dorier · d83b9f11 · Jan 21, 2026 · 9 filesMessage 57 · ThinTriage 0Details
Commit message · Nicolas Dorier
Refactor: Move boltcards related code to plugins
57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
✓ Subject identifies a change✓ Uses a recognizable type or scope✓ Provides an explanatory body✓ Mentions testing or verification
AI review queuedAvoid NRE in ErrorControllerby Nicolas Dorier · 6fade98a · Jan 20, 2026 · 1 fileMessage 35 · OpaqueLow 34Details
Commit message · Nicolas Dorier
Avoid NRE in ErrorController
35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Low 34/100
This commit fixes a potential server crash in BTCPay Server's error-handling page. Previously, if a visitor requested a non-HTML error page without specifying a status code, the code could try to use a missing value and throw a NullReferenceException. The change supplies a fallback 500 status code and renames a loop variable to avoid shadowing. It is a small defensive fix rather than a clear-cut exploitable vulnerability.
Lower-priorityTests: Fix flaky tests when opening app link from app settingsby Nicolas Dorier · e438f997 · Jan 20, 2026 · 3 filesMessage 60 · AdequateTriage 0Details
Commit message · Nicolas Dorier
Tests: Fix flaky tests when opening app link from app settings
60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
Lower-priorityRefactor: Move NFC views in its Pluginsby Nicolas Dorier · 3f26397b · Jan 20, 2026 · 3 filesMessage 57 · ThinTriage 0Details
Commit message · Nicolas Dorier
Refactor: Move NFC views in its Plugins
57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Security candidateRefactor: Move PoS views in PoS plugin folderby Nicolas Dorier · e09133e0 · Jan 20, 2026 · 12 filesMessage 57 · ThinInformational 15Details
Commit message · Nicolas Dorier
Refactor: Move PoS views in PoS plugin folder
57/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Why it was queued
signing or wallet pathboot or update path
AI analysis · Informational 15/100
This commit is a straightforward code reorganization: it moves the Point of Sale (PoS) user-interface files from a shared views folder into the dedicated Point of Sale plugin folder, and updates the corresponding file paths in the controller and plugin registration. There is no functional change to how the application behaves, and no security fix or vulnerability is introduced.
Lower-priorityTests: Migrate Payjoin tests (#7119)by Nicolas Dorier · 74e5a8d3 · Jan 20, 2026 · 4 filesMessage 63 · AdequateTriage 0Details
Commit message · Nicolas Dorier
Tests: Migrate Payjoin tests (#7119)
63/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
AI review queuedFix testby Nicolas Dorier · 29eaf670 · Jan 20, 2026 · 1 fileMessage 0 · OpaqueInformational 15Details
Commit message · Nicolas Dorier
Fix test
0/100 · OpaqueMessage clarity
✓ Mentions testing or verification! 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 changes a single character in a test file, flipping a comparison operator from '==' to '!='. It is a test-only fix that corrects the logic of an assertion checking whether a UI alert appears. There is no change to production code and no security relevance.
Lower-priority(Refactor) : Converted Selenium test for `CanCreatePayRequest` and Others to playwright (#7006)by Abhijay Jain · 53beab6e · Jan 20, 2026 · 2 filesMessage 91 · StrongTriage 0Details
Commit message · Abhijay Jain
(Refactor) : Converted Selenium test for `CanCreatePayRequest` and Others to playwright (#7006)
* (Test):Converted/Added Playwright Test for CanCreateCrowdfundingApp
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
* (Refactor): Removed Selenium Test for CanCreateCrowdfundingApp
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
* (Test):Converted/Added Playwright Test for CanCreatePayRequest
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
* (Refactor): Removed Selenium Test for CanCreatePayRequest
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
* refactor: updated tests
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
* refactor: added wait for view app
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
* refactor: addressed code rabbit suggestions
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
---------
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
91/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
Lower-priorityTests: More info if AssertNoCallToAction failsby Nicolas Dorier · b8401cdc · Jan 20, 2026 · 1 fileMessage 55 · ThinTriage 0Details
Commit message · Nicolas Dorier
Tests: More info if AssertNoCallToAction fails
55/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
* feat: allow label manager to scope suggestions by linked type
* fix: render unlabeled payment requests as "Unlabeled" in reports
* feat: apply linked type scoped labels to EditPaymentRequest view
* feat: add store-scoped label tables
* test: cover store-scoped label migration
* feat: add StoreLabelRepository
* feat: support store-scoped LabelManager
* refactor: payment requests use store-scoped labels
* fix: adapt ReservedAddresses to labelmanager event
* refactor: coderabbit nits
* refactor: type-scoped store labels schema with snake_case
* refactor: persist label colors as column
* refactor: cleanup label links and usage
* refactor: simplify label repository ops
* fix: secure label updates and ensure pgcrypto
* test: stabilize CanUpgradeAndDowngrade
* refactor: coderabbit nits
* feat: use snake_case schema and case-insensitive label merge
* test: add playwright coverage for store label casing
* refactor: use snake_case schema and indexed case-insensitive label lookups
91/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Mentions testing or verification✓ Links an issue, advisory, or supporting reference
Why it was queued
update trustsigning or wallet path
AI analysis · Low 29/100
This commit refactors how BTCPay Server stores labels for payment requests, moving them from a wallet-based graph system into new store-scoped database tables. It also adds a new API endpoint for updating store-scoped labels. The changes are primarily a feature/refactoring effort, but they include security-relevant hardening such as CSRF token validation on the new update endpoint, a check that the target store exists and matches the route, and restrictions preventing deletion or renaming of reserved system label types. A migration copies existing payment-request labels into the new tables and removes the old wallet-graph entries.
AI review queuedfix: scan-qrcode icon alignment (#7116)by Carlos Marques · 36e2e6ee · Jan 20, 2026 · 1 fileMessage 65 · AdequateInformational 15Details
Commit message · Carlos Marques
fix: scan-qrcode icon alignment (#7116)
65/100 · AdequateMessage clarity
✓ 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: security-sensitive path
AI analysis · Informational 15/100
This commit is a purely cosmetic fix for a QR-code scan button icon in the wallet send page. It adjusts CSS margins and adds flexbox centering so the icon lines up correctly. There is no security relevance.