Skip to content

Commit b9a89db

Browse files
foleydangclaude
andcommitted
fix: add null check for list param in Files and SessionEvents
Files.listAsync and SessionEvents.listAsync called param.toQueryString() without checking if param was null, causing NPEs. All other listAsync methods already had this guard. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent bf83d62 commit b9a89db

2 files changed

Lines changed: 6 additions & 0 deletions

File tree

src/main/java/com/alibaba/dashscope/agentstudio/resource/Files.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -146,6 +146,9 @@ public CompletableFuture<AgentStudioFile> retrieveAsync(
146146
}
147147

148148
public CompletableFuture<CursorPage<AgentStudioFile>> listAsync(FileListParam param) {
149+
if (param == null) {
150+
return AsyncHelper.failedFuture(new InputRequiredException("param is required!"));
151+
}
149152
String query = param.toQueryString();
150153
String path = query.isEmpty() ? "files" : "files?" + query;
151154
GeneralServiceOption opt = AgentStudioConstants.newServiceOption(HttpMethod.GET, path, baseUrl);

src/main/java/com/alibaba/dashscope/agentstudio/resource/SessionEvents.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,6 +98,9 @@ public CompletableFuture<CursorPage<Message>> listAsync(
9898
if (sessionId == null || sessionId.isEmpty()) {
9999
return AsyncHelper.failedFuture(new InputRequiredException("sessionId is required!"));
100100
}
101+
if (param == null) {
102+
return AsyncHelper.failedFuture(new InputRequiredException("param is required!"));
103+
}
101104
String query = param.toQueryString();
102105
String path = StringUtils.format("sessions/%s/events", sessionId);
103106
GeneralServiceOption opt =

0 commit comments

Comments
 (0)