Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Mold - 보안인증] log4j 업그레이드 #325

Conversation

stardom3645
Copy link

PR 설명

  • 보안인증을 위한 log4j 업그레이드
    • log4j 1.2 -> log4j 2.19(cloudstack) -> log4j 2.20(ablestack-diplo)

변경 구분

  • 잠재적 기능/오류 개선 (기존의 기능에 잠재되어 있는 오류 또는 다른 기능에 영향을 미칠 기능의 개선)
  • 새로운 기능 (다른 기능에 영향을 미치지 않는 새 기능)
  • 버그 수정 (이슈에 보고된 버그에 대한 수정으로 다른 기능에 영향을 미치지 않음)
  • 기능 개선 (기존 기능에 대한 개선으로 다른 기능에 영향을 미치지 않음)
  • 코드 청소 (코드 재구성 및 청소, 테스트 케이스 추가 등)

기능/개선 규모 또는 버그 심각도

기능/개선 규모

  • 주요 기능/개선
  • 소규모 기능/개선

버그 심각도

  • 매우 심각 (제품 출시/운영을 불가능하게 함)
  • 위험 (사용자에게 자원의 손실을 가져오게 함)
  • 중요 (사용자에게 기능 사용의 불편을 가져오게 함)
  • 보통 (사용자가 문제를 인지할 수 있을 정도임)
  • 미미 (사용자가 인지하지 못할 정도임)

스크린샷

테스트 방법 및 결과

All classes that could have their loggers inherited from their fathers had their own loggers deleted;
Most loggers didn't have to be static, so most of them were normalized so that they wouldn't be;
All loggers are protected now;
Static logger's name are now 'LOGGER';
Non-static logger's name are now 'logger';
New class DbUpgradeAbstractImpl created so that all Upgraders extend it and inherit its logger
…variable that was mistakenly renamed on the normalization commit
JoaoJandre and others added 25 commits July 4, 2023 11:38
…ade-fifth

# Conflicts:
#	api/src/main/java/org/apache/cloudstack/api/command/user/iso/GetUploadParamsForIsoCmd.java
#	debian/rules
#	engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java
#	framework/quota/src/main/java/org/apache/cloudstack/quota/QuotaAlertManagerImpl.java
#	packaging/centos7/cloud.spec
#	packaging/centos8/cloud.spec
#	plugins/hypervisors/kvm/src/main/java/com/cloud/ha/KVMInvestigator.java
#	plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/BridgeVifDriver.java
#	plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/KVMHAChecker.java
#	plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/KVMHAMonitor.java
#	plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/KVMHAVMActivityChecker.java
#	plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
#	plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtFenceCommandWrapper.java
#	plugins/hypervisors/kvm/src/main/java/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java
#	plugins/hypervisors/kvm/src/main/java/org/apache/cloudstack/kvm/ha/KVMHAProvider.java
#	plugins/integrations/cloudian/src/main/java/org/apache/cloudstack/cloudian/client/CloudianUtils.java
#	plugins/outofbandmanagement-drivers/ipmitool/src/main/java/org/apache/cloudstack/outofbandmanagement/driver/ipmitool/IpmitoolOutOfBandManagementDriver.java
#	server/src/main/java/com/cloud/agent/manager/allocator/impl/FirstFitAllocator.java
#	server/src/main/java/com/cloud/api/ApiServer.java
#	server/src/main/java/com/cloud/api/query/ViewResponseHelper.java
#	server/src/main/java/com/cloud/server/ConfigurationServerImpl.java
#	server/src/main/java/com/cloud/user/AccountManagerImpl.java
#	server/src/main/java/com/cloud/user/PasswordPolicyImpl.java
#	server/src/main/java/org/apache/cloudstack/outofbandmanagement/OutOfBandManagementServiceImpl.java
#	utils/src/main/java/org/apache/cloudstack/utils/process/ProcessRunner.java
@github-actions
Copy link

github-actions bot commented Oct 4, 2023

This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch.

@stardom3645 stardom3645 deleted the hwryu-log4j-upgrade-fifth branch November 17, 2023 01:37
jschoiRR pushed a commit that referenced this pull request Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants