frankgh commented on code in PR #239:
URL: https://github.com/apache/cassandra-sidecar/pull/239#discussion_r2263757848


##########
client-common/src/main/java/org/apache/cassandra/sidecar/common/ApiEndpointsV1.java:
##########
@@ -157,6 +157,11 @@ public final class ApiEndpointsV1
     public static final String LIVE_MIGRATION_FILE_TRANSFER_API = 
LIVE_MIGRATION_FILES_API + "/:" + DIR_TYPE_PARAM
                                                                   + "/:" + 
DIR_INDEX_PARAM + "/*";
 
+    public static final String OPENAPI_ROUTE = "/openapi";
+    public static final String OPENAPI_JSON_ROUTE = "/openapi.json";
+    public static final String OPENAPI_YAML_ROUTE = "/openapi.yaml";
+    public static final String OPENAPI_HTML_ROUTE = "/openapi.html";

Review Comment:
   NIT
   ```suggestion
       public static final String OPENAPI_JSON_ROUTE = OPENAPI_ROUTE + ".json";
       public static final String OPENAPI_YAML_ROUTE = OPENAPI_ROUTE + ".yaml";
       public static final String OPENAPI_HTML_ROUTE = OPENAPI_ROUTE + ".html";
   ```



##########
NOTICE.txt:
##########
@@ -4,3 +4,6 @@ Copyright 2023- The Apache Software Foundation
 This product includes software developed by The Apache Software
 Foundation (http://www.apache.org/).
 
+This product includes the Jakarta RESTful Web Services API (jakarta.ws.rs-api),
+licensed under the Eclipse Public License 2.0 and/or the GNU General Public 
License v2.0 with the Classpath Exception.

Review Comment:
   Since we are choosing the EPL license, let's change the verbiage to this
   ```suggestion
   licensed under the Eclipse Public License 2.0.
   ```



##########
server/src/main/resources/openapi-ui.html:
##########
@@ -0,0 +1,62 @@
+<!-- HTML for static distribution bundle build -->
+<!DOCTYPE html>
+<!--
+  Licensed to the Apache Software Foundation (ASF) under one or more
+  contributor license agreements.  See the NOTICE file distributed with
+  this work for additional information regarding copyright ownership.
+  The ASF licenses this file to You under the Apache License, Version 2.0
+  (the "License"); you may not use this file except in compliance with
+  the License.  You may obtain a copy of the License at
+
+      http://www.apache.org/licenses/LICENSE-2.0
+
+  Unless required by applicable law or agreed to in writing, software
+  distributed under the License is distributed on an "AS IS" BASIS,
+  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+  See the License for the specific language governing permissions and
+  limitatins under the License.

Review Comment:
   ```suggestion
     limitations under the License.
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org

Reply via email to