BT
← All projectsBTCPay Server

BTCPay Server

Free, open-source, self-hosted Bitcoin payment processor supporting on-chain and Lightning payments.

BitcoinLightning NetworkPayment infrastructureNormal
Repository coverage

660 commits in the local evidence base

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.

85security candidates223second-pass queue308AI analyses
93commits · 30 days
117commits · 60 days
314commits · 180 days
633commits · 365 days
Backfill bands
Aug 8 → Feb 9247 seen22 candidatesComplete
Feb 9 → Jun 9197 seen39 candidatesComplete
Jun 9 → Jul 954 seen0 candidatesComplete
Jul 9 → Aug 865 seen16 candidatesComplete
Commit communication

Does the history explain itself?

Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.

53/100 average clarity
80Strong · 80–100
148Adequate · 60–79
276Thin · 40–59
156Opaque · 0–39
8security candidates with opaque commit messaging
Read the scoring rubric →
Developer activity

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
Nicolas Dorier43648226346
Cerberus622290
ndeet111133
rockstardev33612049
Abhijay Jain26610087
Chukwuleta Tobechi25610068
thgO.O32512066
dstrukt625062
Tim522074
Atharva Borade711067
Pavlenex412065
psam21312068
Analysis record

Published AI watches

Last scanned 4 minutes ago

Informational 15 AI analysisMessage 58 · Thin
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Containerize the server settings views into sections (#7501)

This commit is a user-interface redesign, not a security fix. It wraps existing server settings pages into consistent visual sections and adds short explanatory subtitles. No code handling payments, authentication, permissions, or data val…

97357e8cby dstrukt+255−11230 files
No security note in commit
Informational 15 AI analysisMessage 53 · Thin
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Rename global search keywords to aliases (#7586)

This commit simply renames the search feature's 'Keywords' field to 'Aliases' across the BTCPay Server codebase. It is a non-functional refactoring that does not change any security behavior, access controls, or data handling. Backward com…

3af94084by Nicolas Dorier+96−7720 files
No security note in commit
Moderate 58 AI analysisMessage 36 · Opaque
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

fix confirm (#7570)

This commit adds a missing safety check in the API key authorization flow. Previously, if a user clicked a 'confirm' action for an API key that no longer existed, the code would try to use a null (non-existent) key object, which could caus…

Null dereference / NullReferenceException preventedMissing validation of repository return valueUser-facing authorization flow hardening
bd7f91e2by monasco+9−01 file
No security note in commit
Informational 3 AI analysisMessage 0 · Opaque
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Bump libs

This commit simply updates four external software library (NuGet package) versions to newer patch releases. There are no code changes shown, and the commit message gives no indication that any security issue is being fixed. It looks like a…

48c57151by Nicolas Dorier+5−54 files
No security note in commit
Low 35 AI analysisMessage 36 · Opaque
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

fix notfound (#7572)

This commit changes how BTCPay Server handles a missing user when an administrator tries to resend a verification email. Previously, the code threw a generic internal error (ApplicationException), which could expose internal details or pro…

Replaces thrown ApplicationException with NotFound() for missing userReduces information leakage via exception message containing userIdImproves HTTP semantics (404 instead of 500-class error)
491cf201by monasco+1−31 file
No security note in commit
Informational 15 AI analysisMessage 28 · Opaque
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Add PushNuget.sh

This commit adds a simple build-and-publish helper script for the project's NuGet package. It packages the BTCPayServer.Client library, pushes it to the public NuGet registry using an API key from an environment variable, and creates a mat…

24f5054eby Nicolas Dorier+16−01 file
No security note in commit
Low 36 AI analysisMessage 58 · Thin
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Encode payjoin errors in wallet status messages (#7567)

This commit fixes a potential cross-site scripting (XSS) issue in BTCPay Server's wallet status messages. When a payjoin transaction fails, the server shows a warning message that includes an error string. Previously, that error string was…

HTML content constructed from an external error string without encodingAddition of HtmlEncoder.Default.Encode around user-influenced or third-party error textStatus message rendered as raw Html in the UI
a60f8bf2by Nicolas Dorier+2−11 file
No security note in commit
Low 46 AI analysisMessage 36 · Opaque
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

fix lnurl (#7563)

This commit fixes a bug in BTCPay Server's LNURL feature for pull payments. Previously, if someone requested a LNURL for a pull payment that didn't exist, the code would try to use a null (empty) pull payment object, which could cause the …

Null dereference / missing null check on database/service lookup resultPotential server-side exception (DoS/crash) on crafted LNURL requestInformation disclosure risk if exception details leak stack traces
227912f2by monasco+1−11 file
No security note in commit
Informational 15 AI analysisMessage 63 · Adequate
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Update POS callback authentication guidance (#7566)

This commit only updates user-facing help text in the Point of Sale plugin. It replaces outdated guidance about legacy API keys and Basic authentication with newer guidance about API tokens and the correct REST API endpoint. No code logic,…

045f70a2by Nicolas Dorier+2−22 files
No security note in commit
Informational 15 AI analysisMessage 0 · Opaque
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Fix test

This is a one-line change to a test file. It adjusts an assertion so that the test now expects a database-migrated API key to have a null CreatedAt value instead of a non-null value. There is no production code change and no security relev…

ecfb7990by Nicolas Dorier+1−11 file
No security note in commit
Moderate 56 AI analysisMessage 45 · Thin
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Merge branch 'refact-api-keys'

This commit refactors how BTCPay Server stores and handles API keys. Previously, the secret API key itself was used as the database primary key, meaning the full secret was stored in plaintext and appeared in URLs/revocation endpoints. Now…

API key secrets no longer used as database primary key or in revocation URLsDatabase now stores SHA256 hash of secret rather than plaintext secret for authentication lookupNew ephemeral Key column cleared after 5 minutes by scheduled cleanup
7ec0260bby Nicolas Dorier+368−39129 files
Vendor flagged security relevance
Moderate 60 AI analysisMessage 28 · Opaque
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Harden API key storage

This commit changes how BTCPay Server stores and handles API keys. Previously, the secret API key itself was used as the database primary key and was stored in plaintext. After this change, the database stores a one-way hash of the secret,…

Database now stores SHA-256 hash of API key secret instead of the secret itselfAPI key secret is cleared from database after creation via scheduled cleanup jobPublic management ID (akid_*) is separated from the secret
b1294924by Nicolas Dorier+368−39129 files
Vendor flagged security relevance
Low 32 AI analysisMessage 18 · Opaque
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

bump HtmlSanitizer

This commit updates the HtmlSanitizer library to a newer patch version and reorganizes some account email-change tests. The library bump could fix a security bug in how user-supplied HTML is cleaned, but the commit itself does not say it f…

Dependency version bump of an HTML-sanitization library (HtmlSanitizer)Test-only reorganization around account email change flowsNo explicit security advisory, CVE, or vulnerability description in commit message
f8946b83by Nicolas Dorier+17−332 files
No security note in commit
Moderate 58 AI analysisMessage 50 · Thin
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Require current password for account email changes

This commit adds a security check so that when a user tries to change their email address on their account profile, they must enter their current password. Before this change, an attacker who had already hijacked a logged-in session could …

Account-takeover mitigation: email change now requires password re-authenticationNew model property CurrentPassword with DataType.PasswordController now calls CheckPasswordAsync before applying email change
f681ec7eby Nicolas Dorier+74−137 files
Vendor flagged security relevance
Informational 15 AI analysisMessage 28 · Opaque
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Make a signed commit

This commit changes a single word in a build script's status message, from 'if it is possible' to 'whether it is possible'. There is no functional, security, or behavioral change to the software.

e342c62aby Nicolas Dorier+1−11 file
No security note in commit
Low 35 AI analysisMessage 70 · Adequate
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

fix(plugin-manager): use update lookup for installed plugins (#7536)

This commit changes how BTCPay Server's plugin manager asks the plugin directory for update information. Previously, the server fetched the full catalog of plugins and then filtered locally. Now it sends a list of the plugins actually inst…

Reduces information disclosure to external plugin directory by sending only installed/pending plugin list instead of querying full catalogAdds input validation on plugin update response (null entries, missing identifier/version)Improves handling of disabled and pending plugins in update checks
c617a24fby thgO.O+248−685 files
No security note in commit
Informational 15 AI analysisMessage 38 · Opaque
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Fix flaky test

This commit adds one extra wait/check to an automated test that verifies email rules appear in a web page. It is purely a test-stability fix and does not change any production code, user-facing behavior, or security boundary.

34ce3589by Nicolas Dorier+1−01 file
No security note in commit
Informational 15 AI analysisMessage 38 · Opaque
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Fix github reports in CI

This commit changes a CI test script so that GitHub Actions can write step summary reports to the correct file path inside a Docker container. It is a build/test infrastructure fix with no apparent security relevance.

4208bf71by Nicolas Dorier+15−11 file
No security note in commit
Moderate 66 AI analysisMessage 85 · Strong
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Validate support URL scheme to prevent stored script injection (#7537)

This commit fixes a stored cross-site scripting (XSS) risk in BTCPay Server's store settings. Merchants can set a 'Support URL' that is shown to customers during checkout. Before this fix, an attacker with access to store settings could en…

Stored XSS via javascript: URI in SupportUrlMissing scheme validation on user-supplied URLGreenfield API and UI controller both patched
a6b81460by Chukwuleta Tobechi+19−33 files
Vendor flagged security relevance
Moderate 61 AI analysisMessage 58 · Thin
BT BTCPay ServerBTCPay Server BitcoinLightning NetworkPayment infrastructure

Prevent monetization from overriding administrator account lockout (#7523)

This change fixes a bug where BTCPay Server's paid-subscription ('monetization') system could automatically re-enable an administrator account that had been manually disabled. Previously, when a subscription renewed or was bypassed, the mo…

Privilege/authorization bypass: automated subsystem overriding an administrative account-disable actionMissing provenance/audit trail in security-sensitive state change (account disabled flag)Business-logic flaw in subscription lifecycle interacting with identity lockout
0e4fc389by Chukwuleta Tobechi+28−64 files
No security note in commit
Repository ledger

Explore captured commits

Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.

AI review queuedBump dotnetby Nicolas Dorier · 5e8c3d73 · Jun 10, 2026 · 2 filesMessage 0 · OpaqueLow 31Details
Commit message · Nicolas Dorier

Bump dotnet

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 · Low 31/100

This commit updates the .NET runtime and SDK versions used to build and run BTCPay Server. It is a routine dependency bump, likely to pick up bug fixes and security patches released by Microsoft. The commit itself does not describe any specific vulnerability or attack, but running older .NET versions can expose software to known security flaws.

AI review queuedbump depsby Nicolas Dorier · 41e348f4 · Jun 9, 2026 · 8 filesMessage 0 · OpaqueLow 25Details
Commit message · Nicolas Dorier

bump deps

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 · Low 25/100

This commit updates several third-party software libraries (dependencies) used by BTCPay Server to newer patch versions. It also adds a few new user-interface translation strings and records a bug fix in the changelog. The dependency bumps are routine maintenance that may include undisclosed security fixes from upstream vendors, but the commit itself does not describe any specific vulnerability or attack.

AI review queuedUpdate changelogby Nicolas Dorier · 91ab7e72 · Jun 9, 2026 · 1 fileMessage 18 · OpaqueInformational 15Details
Commit message · Nicolas Dorier

Update changelog

18/100 · OpaqueMessage clarity
✓ Subject identifies a change! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
documentation-only discountsecond-pass: opaque commit message
AI analysis · Informational 15/100

This commit only updates the project's changelog file, adding one line describing a UI fix for an invoice date filter widget. There is no code change and no security relevance in the diff itself.

AI review queuedUpdate Changelogby Nicolas Dorier · 7517f92a · Jun 9, 2026 · 1 fileMessage 18 · OpaqueInformational 15Details
Commit message · Nicolas Dorier

Update Changelog

18/100 · OpaqueMessage clarity
✓ Subject identifies a change! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
documentation-only discountsecond-pass: opaque commit message
AI analysis · Informational 15/100

This commit only updates the project's changelog file, adding a single line describing a user-interface improvement (inline QR code and copy button for a Store ID). There is no code change, no security fix, and no functional change to the software.

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
Why it was queued
translation-only discountsecond-pass: opaque commit message
AI analysis · Informational 15/100

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.

AI review queuedfeat: Add multisig wallet setup (#7218)by thgO.O · e48a8438 · Jun 8, 2026 · 56 filesMessage 65 · AdequateInformational 15Details
Commit message · thgO.O

feat: Add multisig wallet setup (#7218)

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: 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
Why it was queued
second-pass: broader security terminologysecond-pass: opaque commit message
AI analysis · Low 35/100

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.

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.

AI review queuedfeat(wallets): add transaction search and date filters. (#7133)by Sup3rlativ3 · 7b314873 · May 25, 2026 · 8 filesMessage 100 · StrongInformational 22Details
Commit message · Sup3rlativ3

feat(wallets): add transaction search and date filters. (#7133)

* feat(wallets): add transaction search and date filters

* chore(i18n): mark wallet date label

* test(wallets): cover transaction search filters

* fix(wallets): use nolabel filter

* test(wallets): update nolabel filter assertion

* test(wallets): relax export url assertion

* fix(wallets): preserve filters on custom range

* fix(wallet): align transaction filters with invoice pattern

* fix(wallet): resolve export menu merge conflict

* fix(wallet): handle custom date bounds and case-insensitive direction filters

* fix(wallet): add all-time date reset and end-date label

* chore(ci): rerun pipeline after transient apt mirror failure

* fix(wallet): repair transaction direction filter and tests

* fix(wallet): make transactions toolbar responsive

* fix(wallet): restore direct user id lookup

* chore(ci): retrigger pipeline

* test(wallet): preserve fractional timestamp precision

* fix(wallet): align transaction export actions

* fix(wallet): guard transaction nullability paths

* fix(wallet): align date filters with invoice timezone flow

* test(wallet): cover clear all filter reset

* test(wallet): cover label filter search text preservation

* fix(wallet): submit transaction search on enter

* test(wallet): cover date preset search text preservation

* test(wallet): cover no-label transaction filter

* test(wallet): cover combined direction and label filters

* test(wallet): cover export link filter state

* test(wallet): cover transaction filter term parsing

* refactor(wallet): drop redundant antiforgery attribute

* fix(wallet): preserve timezone in preset date filters

* fix(wallet): label transaction search input

* fix(wallet): ignore unsupported search filter tokens

* perf(wallet): defer transaction metadata hydration

* 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.

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.

AI review queuedFix corrutped table in 'Enter your xpub' screenby Nicolas Dorier · 2b61a437 · May 18, 2026 · 1 fileMessage 45 · ThinInformational 15Details
Commit message · Nicolas Dorier

Fix corrutped table in 'Enter your xpub' screen

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit fixes a visual layout bug in an HTML table on the 'Enter your xpub' screen. The table had incorrect row span numbers, which made the table display incorrectly. There is no security issue here.

AI review queuedFeature: New global search bar to improve navigation (#7183)by Pavlenex · ea7f7a2f · May 15, 2026 · 46 filesMessage 73 · AdequateInformational 20Details
Commit message · Pavlenex

Feature: New global search bar to improve navigation (#7183)

Co-authored-by: Nicolas Dorier <nicolas.dorier@gmail.com>

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Links an issue, advisory, or supporting reference
Why it was queued
signing or wallet pathsecond-pass: unusually broad changesecond-pass: security-sensitive path
AI analysis · Informational 20/100

This commit adds a new global search bar and reorganizes the top navigation in BTCPay Server. Most of the change is user-interface code (menus, CSS, JavaScript, tests). There is one small security-relevant detail: the new search endpoint returns a list of internal URLs and checks whether the current user is authorized for each one, but it also accepts an arbitrary storeId parameter and queries the store with the current user's permissions. The commit does not describe itself as a security fix, and no vulnerability is clearly demonstrated in the diff.

AI review queuedMove code into WalletsPluginby Nicolas Dorier · f2c94dae · May 13, 2026 · 2 filesMessage 35 · OpaqueInformational 15Details
Commit message · Nicolas Dorier

Move code into WalletsPlugin

35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: opaque commit messagesecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit simply moves three helper methods that generate wallet-related URLs from a general file into a new file inside the Wallets plugin folder. There is no change to what the code does, no bug fix, and no security-related change.

AI review queuedfix warningsby Nicolas Dorier · 6b676c7b · May 13, 2026 · 2 filesMessage 18 · OpaqueInformational 11Details
Commit message · Nicolas Dorier

fix warnings

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 11/100

This commit removes nullable markers ('?') from some string parameters and adds explicit controller names to two URL helper methods. It is titled 'fix warnings' and appears to be a routine compiler-warning cleanup. There is no direct evidence in the commit that these changes fix a security vulnerability.

AI review queuedRemove deprecated Shopify Scripts integration (#6608)by Nicolas Dorier · 0e91f371 · May 8, 2026 · 24 filesMessage 58 · ThinInformational 15Details
Commit message · Nicolas Dorier

Remove deprecated Shopify Scripts integration (#6608)

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
signing or wallet pathsecond-pass: unusually broad changesecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit removes the old Shopify integration from BTCPay Server entirely. It deletes all Shopify-related code, settings pages, API clients, and rate-limiting rules. There is no security patch here—this is a feature removal because Shopify deprecated the underlying technology.

AI review queuedfix: standardize paging in WalletTransactions to use ParseListQuery (#7339)by Abhijay Jain · c6fbe4a8 · May 8, 2026 · 3 filesMessage 85 · StrongInformational 16Details
Commit message · Abhijay Jain

fix: standardize paging in WalletTransactions to use ParseListQuery (#7339)

Signed-off-by: Abhijay Jain <Abhijay007j@gmail.com>

85/100 · StrongMessage clarity
✓ Specific, 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
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 16/100

This commit changes how the wallet transactions page reads its paging settings (skip/count). Instead of taking them directly from the URL, it routes them through a shared helper called ParseListQuery that also reads and saves user preferences in a cookie. The change is a code-quality fix to make paging behave consistently with other pages. There is no direct evidence in the commit that this fixes an active security vulnerability, but it removes a place where paging parameters were handled differently from the rest of the application.

AI review queuedMove greenfield wallet controllersby Nicolas Dorier · 74292ad1 · May 7, 2026 · 4 filesMessage 35 · OpaqueInformational 11Details
Commit message · Nicolas Dorier

Move greenfield wallet controllers

35/100 · OpaqueMessage clarity
✓ Descriptive subject! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: opaque commit messagesecond-pass: security-sensitive path
AI analysis · Informational 11/100

This commit simply moves three Greenfield API wallet controller files from one folder to another inside the BTCPay Server codebase and updates their class constructors to use C# primary constructors. The code logic, authorization checks, route URLs, and security behavior appear unchanged. It is a refactoring/organizational change, not a security fix or feature change.

AI review queuedSmall refactorby Nicolas Dorier · 428132cb · May 7, 2026 · 1 fileMessage 18 · OpaqueLow 44Details
Commit message · Nicolas Dorier

Small refactor

18/100 · OpaqueMessage clarity
✓ Subject identifies a change! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: opaque commit messagesecond-pass: security-sensitive path
AI analysis · Low 44/100

This commit refactors authorization checks in two wallet-related controller actions. In one case, the 'clear' command previously required both server-admin role AND store-level wallet-settings permission; now it only requires a server-level policy. In another case, a new switch statement replaces nested if-checks but appears functionally equivalent. The change could be a security fix if the old combined check was overly restrictive, or a security weakening if the new single policy is less protective than the old two-factor check. Without vendor context, the security relevance is uncertain.

AI review queuedMinor refactoringby Nicolas Dorier · ffcc48b7 · May 7, 2026 · 3 filesMessage 18 · OpaqueInformational 15Details
Commit message · Nicolas Dorier

Minor refactoring

18/100 · OpaqueMessage clarity
✓ Subject identifies a change! Too few words to establish purpose! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: opaque commit messagesecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit is a routine code cleanup that converts three C# controller classes from traditional constructor syntax to a newer, shorter primary-constructor syntax. It does not change what the code does, what data it accepts, who can access it, or how it protects wallet operations. There is no visible security fix or behavior change.

AI review queuedbumpby Nicolas Dorier · a3aaba15 · Apr 24, 2026 · 1 fileMessage 0 · OpaqueInformational 15Details
Commit message · Nicolas Dorier

bump

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 changes a single version number in a build file from 2.3.8 to 2.3.9. There is no code change, no bug fix, and no security-related content visible in the diff.

AI review queuedBump 2.3.9by Nicolas Dorier · 33913243 · Apr 24, 2026 · 1 fileMessage 0 · OpaqueInformational 15Details
Commit message · Nicolas Dorier

Bump 2.3.9

0/100 · OpaqueMessage clarity
! Very short subject! No meaningful explanatory body
Why it was queued
documentation-only discountsecond-pass: opaque commit message
AI analysis · Informational 15/100

This commit only updates the changelog file to add release notes for version 2.3.9. It does not change any source code, configuration, or executable files. There is no security-relevant change in the diff itself.

AI review queuedFix: Server not recovering after a plugin crash (#7335)by Nicolas Dorier · e21e1b73 · Apr 24, 2026 · 1 fileMessage 70 · AdequateLow 40Details
Commit message · Nicolas Dorier

Fix: Server not recovering after a plugin crash (#7335)

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
Why it was queued
second-pass: broader security terminology
AI analysis · Low 40/100

This commit fixes a bug where BTCPay Server would not properly recover after a plugin crashed. The server tries to identify which plugin caused an exception by looking at the stack trace. The old code only checked the main assembly name and did not look inside nested/inner exceptions, so it sometimes failed to find the culprit plugin. The fix makes the search walk through inner exceptions and also maps all assemblies loaded by each plugin, not just the plugin's primary assembly. A second small fix prevents malformed plugin commands from crashing the command parser.

AI review queuedfix(API): add storeId-less routes for invoices, payment requests, and pull payments (#7313)by Abhijay Jain · ad6fa344 · Apr 24, 2026 · 21 filesMessage 70 · AdequateModerate 51Details
Commit message · Abhijay Jain

fix(API): add storeId-less routes for invoices, payment requests, and pull payments (#7313)

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
Why it was queued
second-pass: unusually broad change
AI analysis · Moderate 51/100

This commit changes several BTCPay Server API endpoints so that callers no longer need to supply a store ID when working with existing invoices, payment requests, and pull payments. The server now looks up the store automatically from the invoice/payment-request/pull-payment record. The old store-scoped routes are kept alongside the new shorter routes. The change is described as a convenience/cleanup fix, not as a security patch, and the diff does not show any new authorization checks being added.