mirror of
https://github.com/aljazceru/btcpayserver.git
synced 2025-12-19 06:54:19 +01:00
Store centric UI: Part 3 (#3224)
* Set store context in cookie * Fix page id usages in view * Move Pay Button to nav * Move integrations to plugins nav * Store switch links to wallet if present * Test fixes * Nav fixes * Fix altcoin view * Main nav updates * Wallet setttings nav update * Move storeId cookie fallback to cookie auth handler * View fixes * Test fixes * Fix profile check * Rename integrations nav extension point to store-integrations-nav-list * Allow strings for Active page/category for plugins * Make invoice list filter based on store context * Do not set context if we are running authorizer through tag helper * Fix test and unfiltered invoices * Add permission helper for wallet links * Add sanity checks for payment requests and invoices * Store context in home controller * Fix PayjoinViaUI test * Store context for notifications * Minor UI improvements * Store context for userstores and vault controller * Bring back integrations page * Rename notifications nav pages file * Fix user stores controller policies * Controller policy fixes from code review * CookieAuthHandler: Simplify CanViewInvoices case * Revert "Controller policy fixes from code review" This reverts commit 97e8b8379c2f2f373bac15a96632d2c8913ef4bd. * Simplify LayoutSimple * Fix CanViewInvoices condition Co-authored-by: Kukks <evilkukka@gmail.com>
This commit is contained in:
@@ -1,8 +1,9 @@
|
||||
using System;
|
||||
using System.Threading.Tasks;
|
||||
using BTCPayServer.Abstractions.Constants;
|
||||
using BTCPayServer.Client;
|
||||
using BTCPayServer.Controllers;
|
||||
using BTCPayServer.Data;
|
||||
using BTCPayServer.PaymentRequest;
|
||||
using BTCPayServer.Services.Apps;
|
||||
using BTCPayServer.Services.Invoices;
|
||||
using BTCPayServer.Services.PaymentRequests;
|
||||
@@ -37,28 +38,32 @@ namespace BTCPayServer.Security
|
||||
_invoiceRepository = invoiceRepository;
|
||||
_paymentRequestRepository = paymentRequestRepository;
|
||||
}
|
||||
|
||||
protected override async Task HandleRequirementAsync(AuthorizationHandlerContext context, PolicyRequirement requirement)
|
||||
{
|
||||
if (context.User.Identity.AuthenticationType != AuthenticationSchemes.Cookie)
|
||||
return;
|
||||
|
||||
var isAdmin = context.User.IsInRole(Roles.ServerAdmin);
|
||||
switch (requirement.Policy)
|
||||
{
|
||||
case Policies.CanModifyServerSettings:
|
||||
if (isAdmin)
|
||||
context.Succeed(requirement);
|
||||
return;
|
||||
}
|
||||
|
||||
var userId = _userManager.GetUserId(context.User);
|
||||
if (string.IsNullOrEmpty(userId))
|
||||
return;
|
||||
|
||||
bool success = false;
|
||||
var isAdmin = context.User.IsInRole(Roles.ServerAdmin);
|
||||
|
||||
AppData app = null;
|
||||
StoreData store = null;
|
||||
InvoiceEntity invoice = null;
|
||||
PaymentRequestData paymentRequest = null;
|
||||
string storeId = context.Resource is string s ? s : _httpContext.GetImplicitStoreId();
|
||||
string storeId;
|
||||
var explicitResource = false;
|
||||
if (context.Resource is string s)
|
||||
{
|
||||
explicitResource = true;
|
||||
storeId = s;
|
||||
}
|
||||
else
|
||||
storeId = _httpContext.GetImplicitStoreId();
|
||||
var routeData = _httpContext.GetRouteData();
|
||||
if (routeData != null)
|
||||
{
|
||||
@@ -81,28 +86,52 @@ namespace BTCPayServer.Security
|
||||
{
|
||||
string payReqId = vPayReqId as string;
|
||||
paymentRequest = await _paymentRequestRepository.FindPaymentRequest(payReqId, userId);
|
||||
storeId ??= paymentRequest?.StoreDataId;
|
||||
if (storeId == null)
|
||||
{
|
||||
storeId = paymentRequest?.StoreDataId;
|
||||
}
|
||||
else if (paymentRequest?.StoreDataId != storeId)
|
||||
{
|
||||
paymentRequest = null;
|
||||
}
|
||||
}
|
||||
// resolve from invoice
|
||||
if (routeData.Values.TryGetValue("invoiceId", out var vInvoiceId))
|
||||
{
|
||||
string invoiceId = vInvoiceId as string;
|
||||
invoice = await _invoiceRepository.GetInvoice(invoiceId);
|
||||
storeId ??= invoice?.StoreId;
|
||||
if (storeId == null)
|
||||
{
|
||||
storeId = invoice?.StoreId;
|
||||
}
|
||||
else if (invoice?.StoreId != storeId)
|
||||
{
|
||||
invoice = null;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// store could not be found
|
||||
// Fall back to user prefs cookie
|
||||
if (storeId == null)
|
||||
{
|
||||
return;
|
||||
storeId = _httpContext.GetUserPrefsCookie()?.CurrentStoreId;
|
||||
}
|
||||
|
||||
var store = await _storeRepository.FindStore(storeId, userId);
|
||||
if (storeId != null)
|
||||
{
|
||||
store = await _storeRepository.FindStore(storeId, userId);
|
||||
}
|
||||
|
||||
bool success = false;
|
||||
switch (requirement.Policy)
|
||||
{
|
||||
case Policies.CanModifyServerSettings:
|
||||
if (isAdmin)
|
||||
success = true;
|
||||
break;
|
||||
case Policies.CanViewInvoices:
|
||||
if (store == null || store.Role == StoreRoles.Owner || isAdmin)
|
||||
success = true;
|
||||
break;
|
||||
case Policies.CanModifyStoreSettings:
|
||||
if (store != null && (store.Role == StoreRoles.Owner || isAdmin))
|
||||
success = true;
|
||||
@@ -115,17 +144,31 @@ namespace BTCPayServer.Security
|
||||
if (store != null || isAdmin)
|
||||
success = true;
|
||||
break;
|
||||
case Policies.CanViewProfile:
|
||||
case Policies.CanViewNotificationsForUser:
|
||||
case Policies.CanManageNotificationsForUser:
|
||||
case Policies.CanModifyStoreSettingsUnscoped:
|
||||
if (context.User != null)
|
||||
success = true;
|
||||
break;
|
||||
}
|
||||
|
||||
if (success)
|
||||
{
|
||||
context.Succeed(requirement);
|
||||
_httpContext.SetStoreData(store);
|
||||
|
||||
// cache associated entities if present
|
||||
if (app != null) _httpContext.SetAppData(app);
|
||||
if (invoice != null) _httpContext.SetInvoiceData(invoice);
|
||||
if (paymentRequest != null) _httpContext.SetPaymentRequestData(paymentRequest);
|
||||
if (!explicitResource)
|
||||
{
|
||||
|
||||
if (store != null)
|
||||
{
|
||||
_httpContext.SetStoreData(store);
|
||||
|
||||
// cache associated entities if present
|
||||
if (app != null) _httpContext.SetAppData(app);
|
||||
if (invoice != null) _httpContext.SetInvoiceData(invoice);
|
||||
if (paymentRequest != null) _httpContext.SetPaymentRequestData(paymentRequest);
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user