s3tables: clean up unused code and improve error response formatting

This commit is contained in:
Chris Lu
2026-01-28 09:38:10 -08:00
parent b30631c3b5
commit 08ee4e37d8
2 changed files with 8 additions and 15 deletions

View File

@@ -45,14 +45,9 @@ type S3TablesApiServer struct {
// NewS3TablesApiServer creates a new S3 Tables API server // NewS3TablesApiServer creates a new S3 Tables API server
func NewS3TablesApiServer(s3a *S3ApiServer) *S3TablesApiServer { func NewS3TablesApiServer(s3a *S3ApiServer) *S3TablesApiServer {
filerAddr := ""
if len(s3a.option.Filers) > 0 {
filerAddr = string(s3a.option.Filers[0])
}
return &S3TablesApiServer{ return &S3TablesApiServer{
s3a: s3a, s3a: s3a,
handler: s3tables.NewS3TablesHandler(filerAddr), handler: s3tables.NewS3TablesHandler(),
} }
} }

View File

@@ -24,15 +24,13 @@ const (
// S3TablesHandler handles S3 Tables API requests // S3TablesHandler handles S3 Tables API requests
type S3TablesHandler struct { type S3TablesHandler struct {
filerAddress string
region string region string
accountID string accountID string
} }
// NewS3TablesHandler creates a new S3 Tables handler // NewS3TablesHandler creates a new S3 Tables handler
func NewS3TablesHandler(filerAddress string) *S3TablesHandler { func NewS3TablesHandler() *S3TablesHandler {
return &S3TablesHandler{ return &S3TablesHandler{
filerAddress: filerAddress,
region: DefaultRegion, region: DefaultRegion,
accountID: DefaultAccountID, accountID: DefaultAccountID,
} }
@@ -187,11 +185,11 @@ func (h *S3TablesHandler) writeJSON(w http.ResponseWriter, status int, data inte
func (h *S3TablesHandler) writeError(w http.ResponseWriter, status int, code, message string) { func (h *S3TablesHandler) writeError(w http.ResponseWriter, status int, code, message string) {
w.Header().Set("Content-Type", "application/x-amz-json-1.1") w.Header().Set("Content-Type", "application/x-amz-json-1.1")
w.WriteHeader(status) w.WriteHeader(status)
err := map[string]interface{}{ errorResponse := map[string]interface{}{
"__type": code, "__type": code,
"message": message, "message": message,
} }
json.NewEncoder(w).Encode(err) json.NewEncoder(w).Encode(errorResponse)
} }
// ARN generation helpers // ARN generation helpers