varnitgoyal95/deployment-portal-be
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

INFRA-3970 | Dhruv | use createdBy user

Browse Source
This commit is contained in:
dhruvjoshi
2024-12-20 01:15:46 +05:30
parent acae625f79
commit 237ed985ec
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/main/java/com/navi/infra/portal/v2/changerequest/service/ChangeRequestServiceImpl.java
View File

@@ -254,7 +254,8 @@ public class ChangeRequestServiceImpl implements ChangeRequestService {
"Change request: {} is approved. Initiated merging with current version of the manifest",
changeRequestId);
final var manifest = manifestService.fetchById(changeRequest.getManifestId());
final var manifest = manifestService.fetchByIdWithoutAuthorization(
changeRequest.getManifestId());
try {
changeRequestSlackService.handleChangeRequestUpdate(manifest, changeRequest);
} catch (Exception e) {
@@ -266,7 +267,7 @@ public class ChangeRequestServiceImpl implements ChangeRequestService {
final var mergedManifest = objectMapper.convertValue(manifestMap, Manifest.class);
SecurityContext originalContext = SecurityContextHolder.getContext();
var crCreatorContext = authUtil.getSecurityContextForUser(changeRequest.getUpdatedBy());
var crCreatorContext = authUtil.getSecurityContextForUser(changeRequest.getCreatedBy());
try {
SecurityContextHolder.setContext(crCreatorContext);
manifestService.createOrUpdate(mergedManifest, true);