Skip to content

[Log Agent] fix: handle potential null value before calling toUpperCase() in TestErrorContro#46

Closed
SangkiHan wants to merge 1 commit into
mainfrom
log-agent/fix-record-381
Closed

[Log Agent] fix: handle potential null value before calling toUpperCase() in TestErrorContro#46
SangkiHan wants to merge 1 commit into
mainfrom
log-agent/fix-record-381

Conversation

@SangkiHan

Copy link
Copy Markdown
Member

🤖 Log Agent 자동 분석

에러 원인: 요청 파라미터 또는 변수가 null인 상태에서 .toUpperCase()를 호출하여 발생한 NullPointerException입니다.

수정 내용: Optional을 사용하거나 조건문을 추가하여 값이 null인 경우를 처리하도록 수정합니다.

✅ 자동 수정 적용: src/main/java/com/puppynoteserver/global/TestErrorController.java

Before

public String triggerNpe(@RequestParam(name = "val") String val)
        {
            return val.toUpperCase();
        }

After

public String triggerNpe(@RequestParam(name = "val") String val)
        {
            return (val != null) ? val.toUpperCase() : "";
        }

Record ID: #381 | 트리거: NullPointerException: Cannot invoke "String.toUpperCase()" because "value" is null

@SangkiHan SangkiHan closed this Jul 2, 2026
@SangkiHan SangkiHan deleted the log-agent/fix-record-381 branch July 2, 2026 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant