diff --git a/fineract-provider/src/main/java/org/apache/fineract/cob/service/InlineLoanCOBExecutorServiceImpl.java b/fineract-provider/src/main/java/org/apache/fineract/cob/service/InlineLoanCOBExecutorServiceImpl.java index b8e10bfaa..2c50ec66a 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/cob/service/InlineLoanCOBExecutorServiceImpl.java +++ b/fineract-provider/src/main/java/org/apache/fineract/cob/service/InlineLoanCOBExecutorServiceImpl.java @@ -42,6 +42,7 @@ import org.apache.fineract.infrastructure.jobs.domain.CustomJobParameter; import org.apache.fineract.infrastructure.jobs.domain.CustomJobParameterRepository; import org.apache.fineract.infrastructure.jobs.exception.JobNotFoundException; import org.apache.fineract.infrastructure.jobs.service.InlineExecutorService; +import org.apache.fineract.infrastructure.security.service.PlatformSecurityContext; import org.apache.fineract.infrastructure.springbatch.SpringBatchJobConstants; import org.jetbrains.annotations.NotNull; import org.springframework.batch.core.BatchStatus; @@ -77,6 +78,7 @@ public class InlineLoanCOBExecutorServiceImpl implements InlineExecutorService, private final JobExplorer jobExplorer; private final TransactionTemplate transactionTemplate; private final CustomJobParameterRepository customJobParameterRepository; + private final PlatformSecurityContext context; private Gson gson; @@ -168,10 +170,14 @@ public class InlineLoanCOBExecutorServiceImpl implements InlineExecutorService, } private boolean isLockOverrulable(LoanAccountLock loanAccountLock) { - if (LockOwner.LOAN_COB_PARTITIONING.equals(loanAccountLock.getLockOwner())) { + if (LockOwner.LOAN_COB_PARTITIONING.equals(loanAccountLock.getLockOwner()) || isBypassUser()) { return true; } else { return StringUtils.isNotBlank(loanAccountLock.getError()); } } + + private boolean isBypassUser() { + return context.getAuthenticatedUserIfPresent().isBypassUser(); + } } diff --git a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/jobs/filter/LoanCOBApiFilter.java b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/jobs/filter/LoanCOBApiFilter.java index dea636aa0..08e33e286 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/infrastructure/jobs/filter/LoanCOBApiFilter.java +++ b/fineract-provider/src/main/java/org/apache/fineract/infrastructure/jobs/filter/LoanCOBApiFilter.java @@ -35,6 +35,7 @@ import lombok.RequiredArgsConstructor; import org.apache.commons.lang3.StringUtils; import org.apache.fineract.cob.service.LoanAccountLockService; import org.apache.fineract.infrastructure.core.data.ApiGlobalErrorResponse; +import org.apache.fineract.infrastructure.security.service.PlatformSecurityContext; import org.apache.fineract.portfolio.loanaccount.domain.GLIMAccountInfoRepository; import org.apache.fineract.portfolio.loanaccount.domain.GroupLoanIndividualMonitoringAccount; import org.apache.fineract.portfolio.loanaccount.domain.Loan; @@ -49,6 +50,7 @@ public class LoanCOBApiFilter extends OncePerRequestFilter { private final GLIMAccountInfoRepository glimAccountInfoRepository; private final LoanAccountLockService loanAccountLockService; + private final PlatformSecurityContext context; private static final List HTTP_METHODS = List.of(HttpMethod.POST, HttpMethod.PUT, HttpMethod.DELETE); private static final Function URL_FUNCTION = s -> s.matches("/loans/\\d+.*") || s.matches("/loans/glimAccount/\\d+.*"); @@ -59,7 +61,7 @@ public class LoanCOBApiFilter extends OncePerRequestFilter { @Override protected void doFilterInternal(HttpServletRequest request, HttpServletResponse response, FilterChain filterChain) throws ServletException, IOException { - if (!isOnApiList(request)) { + if (!isOnApiList(request) || isBypassUser()) { proceed(filterChain, request, response); } else { Iterable split = Splitter.on('/').split(request.getPathInfo()); @@ -74,6 +76,10 @@ public class LoanCOBApiFilter extends OncePerRequestFilter { } } + private boolean isBypassUser() { + return context.getAuthenticatedUserIfPresent().isBypassUser(); + } + private boolean isLoanLocked(Long loanId, boolean isGlim) { if (!isGlim) { return loanAccountLockService.isLoanHardLocked(loanId); diff --git a/fineract-provider/src/main/java/org/apache/fineract/useradministration/domain/AppUser.java b/fineract-provider/src/main/java/org/apache/fineract/useradministration/domain/AppUser.java index 0e9133c83..36c1f77ea 100644 --- a/fineract-provider/src/main/java/org/apache/fineract/useradministration/domain/AppUser.java +++ b/fineract-provider/src/main/java/org/apache/fineract/useradministration/domain/AppUser.java @@ -434,6 +434,10 @@ public class AppUser extends AbstractPersistableCustom implements PlatformUser { return this.enabled; } + public boolean isBypassUser() { + return hasAnyPermission("BYPASS_LOAN_WRITE_PROTECTION"); + } + public String getFirstname() { return this.firstname; } diff --git a/fineract-provider/src/main/resources/db/changelog/tenant/changelog-tenant.xml b/fineract-provider/src/main/resources/db/changelog/tenant/changelog-tenant.xml index f652ebe41..f0f92f568 100644 --- a/fineract-provider/src/main/resources/db/changelog/tenant/changelog-tenant.xml +++ b/fineract-provider/src/main/resources/db/changelog/tenant/changelog-tenant.xml @@ -84,4 +84,5 @@ + diff --git a/fineract-provider/src/main/resources/db/changelog/tenant/parts/0065_add_bypass_loan_write_transaction_permission.xml b/fineract-provider/src/main/resources/db/changelog/tenant/parts/0065_add_bypass_loan_write_transaction_permission.xml new file mode 100644 index 000000000..c411b5ec4 --- /dev/null +++ b/fineract-provider/src/main/resources/db/changelog/tenant/parts/0065_add_bypass_loan_write_transaction_permission.xml @@ -0,0 +1,34 @@ + + + + + + + + + + + + + diff --git a/fineract-provider/src/test/java/org/apache/fineract/infrastructure/jobs/filter/LoanCOBApiFilterTest.java b/fineract-provider/src/test/java/org/apache/fineract/infrastructure/jobs/filter/LoanCOBApiFilterTest.java index de7bf1ce1..216c670a4 100644 --- a/fineract-provider/src/test/java/org/apache/fineract/infrastructure/jobs/filter/LoanCOBApiFilterTest.java +++ b/fineract-provider/src/test/java/org/apache/fineract/infrastructure/jobs/filter/LoanCOBApiFilterTest.java @@ -31,9 +31,11 @@ import java.util.Collections; import javax.servlet.FilterChain; import javax.servlet.ServletException; import org.apache.fineract.cob.service.LoanAccountLockService; +import org.apache.fineract.infrastructure.security.service.PlatformSecurityContext; import org.apache.fineract.portfolio.loanaccount.domain.GLIMAccountInfoRepository; import org.apache.fineract.portfolio.loanaccount.domain.GroupLoanIndividualMonitoringAccount; import org.apache.fineract.portfolio.loanaccount.domain.Loan; +import org.apache.fineract.useradministration.domain.AppUser; import org.apache.http.HttpStatus; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.extension.ExtendWith; @@ -55,14 +57,34 @@ class LoanCOBApiFilterTest { private LoanAccountLockService loanAccountLockService; @Mock private GLIMAccountInfoRepository glimAccountInfoRepository; + @Mock + private PlatformSecurityContext context; @Test void shouldProceedWhenUrlDoesNotMatch() throws ServletException, IOException { MockHttpServletRequest request = mock(MockHttpServletRequest.class); - given(request.getPathInfo()).willReturn("/jobs/2/inline"); - given(request.getMethod()).willReturn(HTTPMethods.POST.value()); MockHttpServletResponse response = mock(MockHttpServletResponse.class); FilterChain filterChain = mock(FilterChain.class); + + given(request.getPathInfo()).willReturn("/jobs/2/inline"); + given(request.getMethod()).willReturn(HTTPMethods.POST.value()); + + testObj.doFilterInternal(request, response, filterChain); + verify(filterChain, times(1)).doFilter(request, response); + } + + @Test + void shouldProceedWhenUserHasBypassPermission() throws ServletException, IOException { + MockHttpServletRequest request = mock(MockHttpServletRequest.class); + MockHttpServletResponse response = mock(MockHttpServletResponse.class); + FilterChain filterChain = mock(FilterChain.class); + AppUser appUser = mock(AppUser.class); + + given(request.getPathInfo()).willReturn("/jobs/2/inline"); + given(request.getMethod()).willReturn(HTTPMethods.POST.value()); + given(context.getAuthenticatedUserIfPresent()).willReturn(appUser); + given(appUser.isBypassUser()).willReturn(true); + testObj.doFilterInternal(request, response, filterChain); verify(filterChain, times(1)).doFilter(request, response); } @@ -72,10 +94,12 @@ class LoanCOBApiFilterTest { MockHttpServletRequest request = mock(MockHttpServletRequest.class); MockHttpServletResponse response = mock(MockHttpServletResponse.class); FilterChain filterChain = mock(FilterChain.class); + AppUser appUser = mock(AppUser.class); given(request.getPathInfo()).willReturn("/loans/2/charges"); given(request.getMethod()).willReturn(HTTPMethods.POST.value()); given(loanAccountLockService.isLoanHardLocked(2L)).willReturn(false); + given(context.getAuthenticatedUserIfPresent()).willReturn(appUser); testObj.doFilterInternal(request, response, filterChain); verify(filterChain, times(1)).doFilter(request, response); @@ -86,10 +110,12 @@ class LoanCOBApiFilterTest { MockHttpServletRequest request = mock(MockHttpServletRequest.class); MockHttpServletResponse response = mock(MockHttpServletResponse.class); FilterChain filterChain = mock(FilterChain.class); + AppUser appUser = mock(AppUser.class); given(request.getPathInfo()).willReturn("/loans/2/charges"); given(request.getMethod()).willReturn(HTTPMethods.POST.value()); given(loanAccountLockService.isLoanHardLocked(2L)).willReturn(false); + given(context.getAuthenticatedUserIfPresent()).willReturn(appUser); testObj.doFilterInternal(request, response, filterChain); verify(filterChain, times(1)).doFilter(request, response); @@ -101,11 +127,13 @@ class LoanCOBApiFilterTest { MockHttpServletResponse response = mock(MockHttpServletResponse.class); FilterChain filterChain = mock(FilterChain.class); PrintWriter writer = mock(PrintWriter.class); + AppUser appUser = mock(AppUser.class); given(request.getPathInfo()).willReturn("/loans/2/charges"); given(request.getMethod()).willReturn(HTTPMethods.POST.value()); given(loanAccountLockService.isLoanHardLocked(2L)).willReturn(true); given(response.getWriter()).willReturn(writer); + given(context.getAuthenticatedUserIfPresent()).willReturn(appUser); testObj.doFilterInternal(request, response, filterChain); verify(response, times(1)).setStatus(HttpStatus.SC_CONFLICT); @@ -120,6 +148,7 @@ class LoanCOBApiFilterTest { GroupLoanIndividualMonitoringAccount glimAccount = mock(GroupLoanIndividualMonitoringAccount.class); Loan loan = mock(Loan.class); Long loanId = 2L; + AppUser appUser = mock(AppUser.class); given(request.getPathInfo()).willReturn("/loans/glimAccount/2"); given(request.getMethod()).willReturn(HTTPMethods.POST.value()); @@ -128,6 +157,7 @@ class LoanCOBApiFilterTest { given(loan.getId()).willReturn(loanId); given(loanAccountLockService.isLoanHardLocked(loanId)).willReturn(true); given(response.getWriter()).willReturn(writer); + given(context.getAuthenticatedUserIfPresent()).willReturn(appUser); testObj.doFilterInternal(request, response, filterChain); verify(response, times(1)).setStatus(HttpStatus.SC_CONFLICT);