lewis.moe / tangled-core
forked from tangled.org/core
Monorepo for Tangled
fork atom

appview: do not require auth for upgrade banner

the upgrade banner is left empty if the user's session is expired. this
prevents them from randomly being booted to the login page.

Signed-off-by: oppiliappan <me@oppi.li>

oppi.li 06e01bc7 71ae1be6

verified
+5 -1
unified split
+1 -1
appview/state/router.go
··· 116 117 r.Get("/", s.HomeOrTimeline) 118 r.Get("/timeline", s.Timeline) 119 - r.With(middleware.AuthMiddleware(s.oauth)).Get("/upgradeBanner", s.UpgradeBanner) 120 121 // special-case handler for serving tangled.org/core 122 r.Get("/core", s.Core())
··· 116 117 r.Get("/", s.HomeOrTimeline) 118 r.Get("/timeline", s.Timeline) 119 + r.Get("/upgradeBanner", s.UpgradeBanner) 120 121 // special-case handler for serving tangled.org/core 122 r.Get("/core", s.Core())
+4
appview/state/state.go
··· 249 250 func (s *State) UpgradeBanner(w http.ResponseWriter, r *http.Request) { 251 user := s.oauth.GetUser(r) 252 l := s.logger.With("handler", "UpgradeBanner") 253 l = l.With("did", user.Did) 254 l = l.With("handle", user.Handle)
··· 249 250 func (s *State) UpgradeBanner(w http.ResponseWriter, r *http.Request) { 251 user := s.oauth.GetUser(r) 252 + if user == nil { 253 + return 254 + } 255 + 256 l := s.logger.With("handler", "UpgradeBanner") 257 l = l.With("did", user.Did) 258 l = l.With("handle", user.Handle)