Skip to content

Commit 9a9a290

Browse files
committed
server: return 404 when repository is not found
1 parent 201e601 commit 9a9a290

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

server/impl/src/main/java/com/walmartlabs/concord/server/org/project/ProjectRepositoryManager.java

+8-1
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@
3333
import com.walmartlabs.concord.server.policy.PolicyManager;
3434
import com.walmartlabs.concord.server.policy.PolicyUtils;
3535
import com.walmartlabs.concord.server.repository.RepositoryRefresher;
36+
import com.walmartlabs.concord.server.sdk.ConcordApplicationException;
3637
import org.immutables.value.Value;
3738
import org.jooq.DSLContext;
3839
import org.sonatype.siesta.ValidationErrorsException;
@@ -71,7 +72,13 @@ public ProjectRepositoryManager(ProjectAccessManager projectAccessManager,
7172
}
7273

7374
public RepositoryEntry get(UUID projectId, String repositoryName) {
74-
return repositoryDao.get(projectId, repositoryName);
75+
RepositoryEntry r = repositoryDao.get(projectId, repositoryName);
76+
77+
if (r == null) {
78+
throw new ConcordApplicationException("Repository not found: " + repositoryName, Response.Status.NOT_FOUND);
79+
}
80+
81+
return r;
7582
}
7683

7784
public RepositoryEntry get(UUID orgId, String projectName, String repositoryName) {

0 commit comments

Comments
 (0)