Skip to content

Commit b85937b

Browse files
committed
web: guard getExplorePageData against null QueueConfig
getQueueConfig returns null when the queue name is unknown. Dereferencing it immediately caused an NPE that surfaced as an HTTP 500 in the host app. Return a code=1 error response instead. Assisted-By: Claude Code
1 parent d4fb731 commit b85937b

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

rqueue-web/src/main/java/com/github/sonus21/rqueue/web/service/RqueueQDetailServiceImpl.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -397,6 +397,10 @@ public DataViewResponse getExplorePageData(
397397
int itemPerPage) {
398398
QueueConfig queueConfig = rqueueSystemManagerService.getQueueConfig(src);
399399
DataViewResponse response = new DataViewResponse();
400+
if (queueConfig == null) {
401+
response.set(1, "Queue '" + src + "' does not exist");
402+
return response;
403+
}
400404
boolean deadLetterQueue = queueConfig.isDeadLetterQueue(name);
401405
boolean scheduledQueue = queueConfig.getScheduledQueueName().equals(name);
402406
boolean completionQueue = name.equals(queueConfig.getCompletedQueueName());

0 commit comments

Comments
 (0)