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.
AI review queuedfix: reserved addresses label filtering after label changes and initLabelManager errorby thgO.O · fb3e9cc9 · Dec 17, 2025 · 2 filesMessage 62 · AdequateInformational 23Details
Commit message · thgO.O
fix: reserved addresses label filtering after label changes and initLabelManager error
62/100 · AdequateMessage clarity
✓ Specific, 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 pathsecond-pass: security-sensitive path
AI analysis · Informational 23/100
This commit fixes a UI bug in BTCPay Server's wallet address list. Previously, when a user changed labels on a reserved Bitcoin address, the label filter on the page did not update to reflect the new labels, and an error in the label manager initialization could break the feature. The patch makes the address list listen for label-change events and refresh its internal label data so filtering works correctly after edits.
AI review queuedrefactor: improve label manager caching and refreshby thgO.O · 99eb9f00 · Dec 17, 2025 · 2 filesMessage 62 · AdequateInformational 13Details
Commit message · thgO.O
refactor: improve label manager caching and refresh
62/100 · AdequateMessage clarity
✓ Specific, 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 pathsecond-pass: security-sensitive path
AI analysis · Informational 13/100
This commit is a routine code cleanup of the wallet address label picker in BTCPay Server. It rewrites how the page caches and refreshes the list of available labels, removes an old Vue.js event listener, and adds a guard so the script does nothing if the expected HTML element is missing. There is no clear security bug being fixed and no security relevance is stated by the project.
AI review queuedfeat: improve payment requests listing, filters and reportingby thgO.O · adf5f3d9 · Dec 17, 2025 · 8 filesMessage 62 · AdequateInformational 17Details
Commit message · thgO.O
feat: improve payment requests listing, filters and reporting
62/100 · AdequateMessage clarity
✓ Specific, 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 pathsecond-pass: security-sensitive path
AI analysis · Informational 17/100
This commit adds new UI filters, a label system, and a reporting provider for payment requests. It also changes the search query to use a hand-written SQL statement. The change is described as a feature improvement, not a security fix. The SQL query uses Entity Framework's parameterized raw SQL, which generally protects against SQL injection, but the code mixes raw SQL with LINQ and introduces a new label-filtering path. There is no direct evidence in the commit of a vulnerability being fixed or introduced, but the raw SQL and label-filtering logic are worth reviewing carefully because mistakes in those areas can lead to security bugs.
Lower-priorityFix typos in Swagger API documentation (#7029)by César Pineda · aa49322f · Dec 17, 2025 · 9 filesMessage 76 · AdequateTriage 0Details
Commit message · César Pineda
Fix typos in Swagger API documentation (#7029)
This PR corrects several typos found in the OpenAPI/Swagger template files to improve documentation clarity and consistency.
Changes Grammar fixes:
Fixed "Broadcast happen" → "Broadcast happens" in webhook delivery description Fixed "An payment" → "A payment" in InvoicePaymentSettled summary Fixed "Deletes apps" → "Deletes app" in app deletion endpoint Terminology fixes:
Changes to be committed: modified: BTCPayServer/wwwroot/swagger/v1/swagger.template.api-keys.json modified: BTCPayServer/wwwroot/swagger/v1/swagger.template.apps.json modified: BTCPayServer/wwwroot/swagger/v1/swagger.template.lightning.common.json modified: BTCPayServer/wwwroot/swagger/v1/swagger.template.lightning.internal.json modified: BTCPayServer/wwwroot/swagger/v1/swagger.template.lightning.store.json modified: BTCPayServer/wwwroot/swagger/v1/swagger.template.payment-requests.json modified: BTCPayServer/wwwroot/swagger/v1/swagger.template.pull-payments.json modified: BTCPayServer/wwwroot/swagger/v1/swagger.template.webhooks.json
76/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
AI review queuedFix docker-composeby Nicolas Dorier · 0853900a · Dec 17, 2025 · 3 filesMessage 28 · OpaqueInformational 18Details
Commit message · Nicolas Dorier
Fix docker-compose
28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 18/100
This commit updates test-only Docker configuration files. It upgrades the PostgreSQL database image used in automated testing from version 13.13 to 18.1, fixes indentation, and adds a persistent data volume for the test database. These files are part of the project's internal test harness and are not used in production deployments. There is no direct security fix or vulnerability being patched here.
AI review queuedbump postgresby Nicolas Dorier · 2c64f037 · Dec 17, 2025 · 1 fileMessage 18 · OpaqueInformational 15Details
Commit message · Nicolas Dorier
bump postgres
18/100 · OpaqueMessage clarity
✓ Subject identifies a change! 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 updates the PostgreSQL database version used only in BTCPay Server's automated test environment from version 13.23 to 18.1. It also changes the data directory mount path inside the container. There is no indication this affects production systems or fixes a security vulnerability.
AI review queuedbump postgresby Nicolas Dorier · 9e067d11 · Dec 17, 2025 · 1 fileMessage 18 · OpaqueInformational 17Details
Commit message · Nicolas Dorier
bump postgres
18/100 · OpaqueMessage clarity
✓ Subject identifies a change! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 17/100
This commit updates the PostgreSQL database version used only in BTCPay Server's automated test environment from 13.13 to 13.23, and adds a persistent data volume for that test database. It does not change production code. The newer PostgreSQL patch release likely fixes bugs and security issues in the database software itself, but the commit does not address any vulnerability in BTCPay Server's own code.
Lower-priorityDo not log loading of system plugins by defaultby Nicolas Dorier · a3df88df · Dec 17, 2025 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · Nicolas Dorier
Do not log loading of system plugins by default
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Lower-priorityLess logs about HostedServices by defaultby Nicolas Dorier · 79aa853d · Dec 17, 2025 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · Nicolas Dorier
Less logs about HostedServices by default
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI review queuedAdd publish-docker.shby Nicolas Dorier · 2a5ad7c2 · Dec 17, 2025 · 1 fileMessage 28 · OpaqueInformational 15Details
Commit message · Nicolas Dorier
Add publish-docker.sh
28/100 · OpaqueMessage clarity
✓ Subject identifies a change! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100
This commit adds a simple shell script used by BTCPay Server maintainers to automate creating and pushing a Git version tag when publishing Docker images. It does not change application code, user-facing behavior, or any security-sensitive logic. There is nothing in the commit that introduces a vulnerability or fixes one.
Lower-priorityCan add features when creating planby Nicolas Dorier · b2cb085d · Dec 17, 2025 · 6 filesMessage 45 · ThinTriage 0Details
Commit message · Nicolas Dorier
Can add features when creating plan
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Lower-priorityChangelog for 2.3.0 (#7026)by Nicolas Dorier · 335c906b · Dec 16, 2025 · 2 filesMessage 53 · ThinTriage 0Details
Commit message · Nicolas Dorier
Changelog for 2.3.0 (#7026)
53/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
Lower-priorityAdd buyer mailbox to Invoice Placeholders (#7032)by Nicolas Dorier · f0ebe512 · Dec 16, 2025 · 2 filesMessage 53 · ThinTriage 0Details
Commit message · Nicolas Dorier
Add buyer mailbox to Invoice Placeholders (#7032)
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 queuedWebhookTriggerProviders can by asyncby Nicolas Dorier · 3097860f · Dec 15, 2025 · 2 filesMessage 35 · OpaqueInformational 15Details
Commit message · Nicolas Dorier
WebhookTriggerProviders can by async
35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 15/100
This commit is a routine code refactor that lets webhook event providers run their lookup logic asynchronously. It changes a synchronous method to an async one and updates callers to await it. There is no security issue visible in the diff.
Lower-priorityRevert: Fix clipboard copy for fiat amounts and other numeric valuesby Nicolas Dorier · afcad151 · Dec 12, 2025 · 1 fileMessage 62 · AdequateTriage 0Details
Commit message · Nicolas Dorier
Revert: Fix clipboard copy for fiat amounts and other numeric values
62/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope! No meaningful explanatory body
Lower-prioritychore: fix some minor issues in comments (#6859)by jishudashen · 57ba1a79 · Dec 12, 2025 · 2 filesMessage 80 · StrongTriage 0Details
✓ 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
Lower-priorityCheck that events are properly firedby Nicolas Dorier · 4cb8388a · Dec 10, 2025 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · Nicolas Dorier
Check that events are properly fired
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Lower-priorityLoad plans before firing eventsby Nicolas Dorier · 5bf21b32 · Dec 10, 2025 · 1 fileMessage 45 · ThinTriage 0Details
Commit message · Nicolas Dorier
Load plans before firing events
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI review queuedFix flaky CanMonetizeTestby Nicolas Dorier · 04d31b26 · Dec 10, 2025 · 2 filesMessage 35 · OpaqueInformational 23Details
Commit message · Nicolas Dorier
Fix flaky CanMonetizeTest
35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
second-pass: opaque commit message
AI analysis · Informational 23/100
This commit fixes a flaky automated test for BTCPay Server's subscription/monetization feature. The code change removes a guard that previously prevented a subscriber's plan from being restarted if it had already started. The test change waits for a background activation event and reloads the page before checking the UI. On its own, the patch looks like a timing/test-stability fix rather than a clear security fix, but removing the 'already started' guard could, in theory, allow duplicate plan activations or repeated credits if the event handler runs more than once.
Security candidate[Greenfield] Subscription API (#7022)by Nicolas Dorier · 1d3cb757 · Dec 9, 2025 · 48 filesMessage 43 · ThinLow 37Details
Commit message · Nicolas Dorier
[Greenfield] Subscription API (#7022)
43/100 · ThinMessage clarity
✓ Descriptive subject✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
Why it was queued
boot or update pathauthentication path
AI analysis · Low 37/100
This commit introduces a new Greenfield (REST) Subscription API to BTCPay Server, allowing stores to manage subscription offerings, plans, subscribers, credits, and checkout sessions programmatically. It also refactors existing subscription internals, adds new permissions, and makes small related fixes such as handling empty URL paths and adding a null check for invoice IDs. There is no explicit security bug in the diff, but the large new attack surface and a few design choices warrant careful review.
Security candidateRename permissions for offerings (#7021)by Nicolas Dorier · 58c5e9cd · Dec 5, 2025 · 6 filesMessage 53 · ThinInformational 15Details
Commit message · Nicolas Dorier
Rename permissions for offerings (#7021)
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 commit simply renames two permission strings from 'membership' to 'offerings' across the codebase. It is a non-functional refactor that updates labels, constants, authorization checks, and API documentation to match the new terminology. There is no security-relevant change.
Lower-priorityAvoid crashes if wehbook delivery request isn't availableby Nicolas Dorier · d5f9ca9d · Dec 4, 2025 · 6 filesMessage 50 · ThinTriage 0Details
Commit message · Nicolas Dorier
Avoid crashes if wehbook delivery request isn't available
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Lower-priorityBy default optimistic activation should be falseby Nicolas Dorier · 2ab0cc14 · Dec 4, 2025 · 5 filesMessage 45 · ThinTriage 0Details
Commit message · Nicolas Dorier
By default optimistic activation should be false
45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Lower-priorityAvoid crashes if wehbook delivery request isn't availableby Nicolas Dorier · 58c92714 · Dec 4, 2025 · 5 filesMessage 50 · ThinTriage 0Details
Commit message · Nicolas Dorier
Avoid crashes if wehbook delivery request isn't available
50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Lower-priorityPeriodically cleanup invoices, checkout plan and portal sessions from DB (#7018)by Nicolas Dorier · 159221b1 · Dec 3, 2025 · 7 filesMessage 58 · ThinTriage 0Details
Commit message · Nicolas Dorier
Periodically cleanup invoices, checkout plan and portal sessions from DB (#7018)
58/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body