diff options
author | Mike Crute <mike@crute.us> | 2023-08-30 12:37:50 -0700 |
---|---|---|
committer | Mike Crute <mike@crute.us> | 2023-08-30 12:37:50 -0700 |
commit | 2b3566a33ccaf2b6eac16ffad2b049584fb4aac7 (patch) | |
tree | 0862749a51559801d9b600c6ef98475b5422d9f4 | |
parent | 7655a82ee0bff121aec08edcbaa3f799d34453b2 (diff) | |
download | websocket_proxy-2b3566a33ccaf2b6eac16ffad2b049584fb4aac7.tar.bz2 websocket_proxy-2b3566a33ccaf2b6eac16ffad2b049584fb4aac7.tar.xz websocket_proxy-2b3566a33ccaf2b6eac16ffad2b049584fb4aac7.zip |
Fix possible nil panic
-rw-r--r-- | app/controllers/ca.go | 3 | ||||
-rw-r--r-- | app/controllers/proxy.go | 3 |
2 files changed, 6 insertions, 0 deletions
diff --git a/app/controllers/ca.go b/app/controllers/ca.go index c04dcd8..2ad85c5 100644 --- a/app/controllers/ca.go +++ b/app/controllers/ca.go | |||
@@ -65,6 +65,9 @@ func NewCAHandler(cfg CAHandlerConfig) (*CAHandler, error) { | |||
65 | 65 | ||
66 | func (h *CAHandler) authorizeRequest(c echo.Context, certRequest *ssh.Certificate) error { | 66 | func (h *CAHandler) authorizeRequest(c echo.Context, certRequest *ssh.Certificate) error { |
67 | session := middleware.GetAuthorizedSession(c) | 67 | session := middleware.GetAuthorizedSession(c) |
68 | if session == nil { | ||
69 | return fmt.Errorf("No authenticated session found") | ||
70 | } | ||
68 | 71 | ||
69 | user, err := h.Users.Get(c.Request().Context(), session.UserId) | 72 | user, err := h.Users.Get(c.Request().Context(), session.UserId) |
70 | if err != nil { | 73 | if err != nil { |
diff --git a/app/controllers/proxy.go b/app/controllers/proxy.go index 0777c93..bc49585 100644 --- a/app/controllers/proxy.go +++ b/app/controllers/proxy.go | |||
@@ -45,6 +45,9 @@ func getConnectAddr(c echo.Context) string { | |||
45 | 45 | ||
46 | func (h *ProxyHandler) authorizeRequest(c echo.Context) error { | 46 | func (h *ProxyHandler) authorizeRequest(c echo.Context) error { |
47 | session := middleware.GetAuthorizedSession(c) | 47 | session := middleware.GetAuthorizedSession(c) |
48 | if session == nil { | ||
49 | return fmt.Errorf("No authenticated session found") | ||
50 | } | ||
48 | 51 | ||
49 | user, err := h.Users.Get(c.Request().Context(), session.UserId) | 52 | user, err := h.Users.Get(c.Request().Context(), session.UserId) |
50 | if err != nil { | 53 | if err != nil { |