diff options
author | Jaime Martinez <jmartinez@gitlab.com> | 2021-12-07 07:06:39 +0300 |
---|---|---|
committer | Jaime Martinez <jmartinez@gitlab.com> | 2021-12-08 04:21:29 +0300 |
commit | c57f28159a32b325ec54f190b0f13c4676600d42 (patch) | |
tree | e546d558ae3f4d133cfeda2a83de29434c7dc019 /internal/serving | |
parent | 34190011b2f774bc9a3613850a7be9d67c8ecd7e (diff) |
fix: use if statement instead of switch
Diffstat (limited to 'internal/serving')
-rw-r--r-- | internal/serving/disk/reader.go | 20 |
1 files changed, 13 insertions, 7 deletions
diff --git a/internal/serving/disk/reader.go b/internal/serving/disk/reader.go index 2438daf7..f45c9092 100644 --- a/internal/serving/disk/reader.go +++ b/internal/serving/disk/reader.go @@ -277,15 +277,21 @@ func (reader *Reader) serveCustomFile(ctx context.Context, w http.ResponseWriter } func handleRootError(err error, h serving.Handler) bool { - switch err { - case fs.ErrNotExist: + if err == nil { return false - case context.Canceled: + } + + if errors.Is(err, fs.ErrNotExist) { + return false + } + + if errors.Is(err, context.Canceled) { // Handle context.Canceled error as not found exist https://gitlab.com/gitlab-org/gitlab-pages/-/issues/669 - //httperrors.Serve404(h.Writer) - return true - default: - httperrors.Serve500WithRequest(h.Writer, h.Request, "vfs.Root", err) + httperrors.Serve404(h.Writer) return true } + + httperrors.Serve500WithRequest(h.Writer, h.Request, "vfs.Root", err) + return true + } |