forked from TrueCloudLab/certificates
api/render: implemented Error
This commit is contained in:
parent
9aa480a09a
commit
3389e57c48
2 changed files with 57 additions and 66 deletions
|
@ -1,66 +0,0 @@
|
||||||
package api
|
|
||||||
|
|
||||||
import (
|
|
||||||
"encoding/json"
|
|
||||||
"fmt"
|
|
||||||
"net/http"
|
|
||||||
"os"
|
|
||||||
|
|
||||||
"github.com/pkg/errors"
|
|
||||||
|
|
||||||
"github.com/smallstep/certificates/acme"
|
|
||||||
"github.com/smallstep/certificates/api/log"
|
|
||||||
"github.com/smallstep/certificates/authority/admin"
|
|
||||||
"github.com/smallstep/certificates/errs"
|
|
||||||
"github.com/smallstep/certificates/logging"
|
|
||||||
"github.com/smallstep/certificates/scep"
|
|
||||||
)
|
|
||||||
|
|
||||||
// WriteError writes to w a JSON representation of the given error.
|
|
||||||
func WriteError(w http.ResponseWriter, err error) {
|
|
||||||
switch k := err.(type) {
|
|
||||||
case *acme.Error:
|
|
||||||
acme.WriteError(w, k)
|
|
||||||
return
|
|
||||||
case *admin.Error:
|
|
||||||
admin.WriteError(w, k)
|
|
||||||
return
|
|
||||||
case *scep.Error:
|
|
||||||
w.Header().Set("Content-Type", "text/plain")
|
|
||||||
default:
|
|
||||||
w.Header().Set("Content-Type", "application/json")
|
|
||||||
}
|
|
||||||
|
|
||||||
cause := errors.Cause(err)
|
|
||||||
if sc, ok := err.(errs.StatusCoder); ok {
|
|
||||||
w.WriteHeader(sc.StatusCode())
|
|
||||||
} else {
|
|
||||||
if sc, ok := cause.(errs.StatusCoder); ok {
|
|
||||||
w.WriteHeader(sc.StatusCode())
|
|
||||||
} else {
|
|
||||||
w.WriteHeader(http.StatusInternalServerError)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// Write errors in the response writer
|
|
||||||
if rl, ok := w.(logging.ResponseLogger); ok {
|
|
||||||
rl.WithFields(map[string]interface{}{
|
|
||||||
"error": err,
|
|
||||||
})
|
|
||||||
if os.Getenv("STEPDEBUG") == "1" {
|
|
||||||
if e, ok := err.(errs.StackTracer); ok {
|
|
||||||
rl.WithFields(map[string]interface{}{
|
|
||||||
"stack-trace": fmt.Sprintf("%+v", e),
|
|
||||||
})
|
|
||||||
} else if e, ok := cause.(errs.StackTracer); ok {
|
|
||||||
rl.WithFields(map[string]interface{}{
|
|
||||||
"stack-trace": fmt.Sprintf("%+v", e),
|
|
||||||
})
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
if err := json.NewEncoder(w).Encode(err); err != nil {
|
|
||||||
log.Error(w, err)
|
|
||||||
}
|
|
||||||
}
|
|
|
@ -3,12 +3,20 @@ package render
|
||||||
|
|
||||||
import (
|
import (
|
||||||
"encoding/json"
|
"encoding/json"
|
||||||
|
"fmt"
|
||||||
"net/http"
|
"net/http"
|
||||||
|
"os"
|
||||||
|
|
||||||
|
"github.com/pkg/errors"
|
||||||
"google.golang.org/protobuf/encoding/protojson"
|
"google.golang.org/protobuf/encoding/protojson"
|
||||||
"google.golang.org/protobuf/proto"
|
"google.golang.org/protobuf/proto"
|
||||||
|
|
||||||
|
"github.com/smallstep/certificates/acme"
|
||||||
"github.com/smallstep/certificates/api/log"
|
"github.com/smallstep/certificates/api/log"
|
||||||
|
"github.com/smallstep/certificates/authority/admin"
|
||||||
|
"github.com/smallstep/certificates/errs"
|
||||||
|
"github.com/smallstep/certificates/logging"
|
||||||
|
"github.com/smallstep/certificates/scep"
|
||||||
)
|
)
|
||||||
|
|
||||||
// JSON writes the passed value into the http.ResponseWriter.
|
// JSON writes the passed value into the http.ResponseWriter.
|
||||||
|
@ -56,3 +64,52 @@ func ProtoJSONStatus(w http.ResponseWriter, m proto.Message, status int) {
|
||||||
|
|
||||||
// log.EnabledResponse(w, v)
|
// log.EnabledResponse(w, v)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Error encodes the JSON representation of err to w.
|
||||||
|
func Error(w http.ResponseWriter, err error) {
|
||||||
|
switch k := err.(type) {
|
||||||
|
case *acme.Error:
|
||||||
|
acme.WriteError(w, k)
|
||||||
|
return
|
||||||
|
case *admin.Error:
|
||||||
|
admin.WriteError(w, k)
|
||||||
|
return
|
||||||
|
case *scep.Error:
|
||||||
|
w.Header().Set("Content-Type", "text/plain")
|
||||||
|
default:
|
||||||
|
w.Header().Set("Content-Type", "application/json")
|
||||||
|
}
|
||||||
|
|
||||||
|
cause := errors.Cause(err)
|
||||||
|
if sc, ok := err.(errs.StatusCoder); ok {
|
||||||
|
w.WriteHeader(sc.StatusCode())
|
||||||
|
} else {
|
||||||
|
if sc, ok := cause.(errs.StatusCoder); ok {
|
||||||
|
w.WriteHeader(sc.StatusCode())
|
||||||
|
} else {
|
||||||
|
w.WriteHeader(http.StatusInternalServerError)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// Write errors in the response writer
|
||||||
|
if rl, ok := w.(logging.ResponseLogger); ok {
|
||||||
|
rl.WithFields(map[string]interface{}{
|
||||||
|
"error": err,
|
||||||
|
})
|
||||||
|
if os.Getenv("STEPDEBUG") == "1" {
|
||||||
|
if e, ok := err.(errs.StackTracer); ok {
|
||||||
|
rl.WithFields(map[string]interface{}{
|
||||||
|
"stack-trace": fmt.Sprintf("%+v", e),
|
||||||
|
})
|
||||||
|
} else if e, ok := cause.(errs.StackTracer); ok {
|
||||||
|
rl.WithFields(map[string]interface{}{
|
||||||
|
"stack-trace": fmt.Sprintf("%+v", e),
|
||||||
|
})
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if err := json.NewEncoder(w).Encode(err); err != nil {
|
||||||
|
log.Error(w, err)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
Loading…
Reference in a new issue