This is an automated email from the ASF dual-hosted git repository.
sarankk pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/cassandra-sidecar.git
The following commit(s) were added to refs/heads/trunk by this push:
new c5782978 CASSSIDECAR-477: Remove filesytem path from Http response
(#366)
c5782978 is described below
commit c5782978480d259add3c8968d5bb0518a3dd3e50
Author: Saranya Krishnakumar <[email protected]>
AuthorDate: Tue Jun 30 14:21:52 2026 -0700
CASSSIDECAR-477: Remove filesytem path from Http response (#366)
Patch by Saranya Krishnakumar; Reviewed by Paulo Motta, Francisco Guerrero,
Shailaja Koppu, Yifan Cai for CASSSIDECAR-477
---
CHANGES.txt | 1 +
.../cassandra/sidecar/handlers/snapshots/ClearSnapshotHandler.java | 2 +-
.../cassandra/sidecar/handlers/snapshots/ListSnapshotHandler.java | 2 +-
.../cassandra/sidecar/handlers/sstableuploads/SSTableImportHandler.java | 2 +-
.../org/apache/cassandra/sidecar/routes/RingHandlerIntegrationTest.java | 1 -
5 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/CHANGES.txt b/CHANGES.txt
index 5d8611cc..07239ad9 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -1,5 +1,6 @@
0.5.0
-----
+ * Remove filesystem path from Http response (CASSSIDECAR-477)
* Add basic configuration retrieval logic to ConfigurationManager
(CASSSIDECAR-427)
* RPM package is created with wrong version (CASSSIDECAR-472)
* Define storage provider interface and Cassandra implementation to support
durable job tracking (CASSSIDECAR-373)
diff --git
a/server/src/main/java/org/apache/cassandra/sidecar/handlers/snapshots/ClearSnapshotHandler.java
b/server/src/main/java/org/apache/cassandra/sidecar/handlers/snapshots/ClearSnapshotHandler.java
index c96382cc..8c042410 100644
---
a/server/src/main/java/org/apache/cassandra/sidecar/handlers/snapshots/ClearSnapshotHandler.java
+++
b/server/src/main/java/org/apache/cassandra/sidecar/handlers/snapshots/ClearSnapshotHandler.java
@@ -100,7 +100,7 @@ public class ClearSnapshotHandler extends
AbstractHandler<SnapshotRequestParam>
requestParams, remoteAddress, host,
context.request().method(), cause);
if (cause instanceof FileNotFoundException || cause instanceof
NoSuchFileException)
{
- context.fail(wrapHttpException(HttpResponseStatus.NOT_FOUND,
cause.getMessage()));
+ context.fail(wrapHttpException(HttpResponseStatus.NOT_FOUND, "The
requested snapshot was not found"));
}
else
{
diff --git
a/server/src/main/java/org/apache/cassandra/sidecar/handlers/snapshots/ListSnapshotHandler.java
b/server/src/main/java/org/apache/cassandra/sidecar/handlers/snapshots/ListSnapshotHandler.java
index 6e2b9d43..1c2c49eb 100644
---
a/server/src/main/java/org/apache/cassandra/sidecar/handlers/snapshots/ListSnapshotHandler.java
+++
b/server/src/main/java/org/apache/cassandra/sidecar/handlers/snapshots/ListSnapshotHandler.java
@@ -154,7 +154,7 @@ public class ListSnapshotHandler extends
AbstractHandler<SnapshotRequestParam> i
request, remoteAddress, host, context.request().method(),
cause);
if (cause instanceof FileNotFoundException || cause instanceof
NoSuchFileException)
{
- context.fail(wrapHttpException(HttpResponseStatus.NOT_FOUND,
cause.getMessage()));
+ context.fail(wrapHttpException(HttpResponseStatus.NOT_FOUND, "The
requested snapshot was not found"));
}
else
{
diff --git
a/server/src/main/java/org/apache/cassandra/sidecar/handlers/sstableuploads/SSTableImportHandler.java
b/server/src/main/java/org/apache/cassandra/sidecar/handlers/sstableuploads/SSTableImportHandler.java
index bf00b238..1b7db827 100644
---
a/server/src/main/java/org/apache/cassandra/sidecar/handlers/sstableuploads/SSTableImportHandler.java
+++
b/server/src/main/java/org/apache/cassandra/sidecar/handlers/sstableuploads/SSTableImportHandler.java
@@ -149,7 +149,7 @@ public class SSTableImportHandler extends
AbstractHandler<SSTableImportRequestPa
{
logger.error("Upload directory not found for request={},
remoteAddress={}, " +
"instance={}", request, remoteAddress, host, cause);
- context.fail(wrapHttpException(HttpResponseStatus.NOT_FOUND,
cause.getMessage()));
+ context.fail(wrapHttpException(HttpResponseStatus.NOT_FOUND,
"SSTable upload directory was not found"));
}
else if (cause instanceof IllegalArgumentException)
{
diff --git
a/server/src/test/integration/org/apache/cassandra/sidecar/routes/RingHandlerIntegrationTest.java
b/server/src/test/integration/org/apache/cassandra/sidecar/routes/RingHandlerIntegrationTest.java
index 2ae71a6e..4692cc4f 100644
---
a/server/src/test/integration/org/apache/cassandra/sidecar/routes/RingHandlerIntegrationTest.java
+++
b/server/src/test/integration/org/apache/cassandra/sidecar/routes/RingHandlerIntegrationTest.java
@@ -70,7 +70,6 @@ class RingHandlerIntegrationTest extends IntegrationTestBase
JsonObject error = response.bodyAsJsonObject();
assertThat(error.getInteger("code")).isEqualTo(errorCode);
assertThat(error.getString("message")).contains("The keyspace
unknown_ks, does not exist");
-
context.completeNow();
});
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
