From 6c02768f5d945a5f88fc7b361f7d1501333bbdf3 Mon Sep 17 00:00:00 2001 From: Umputun Date: Sun, 6 Jan 2019 17:08:08 -0600 Subject: [PATCH] fix rest log prefix --- backend/app/rest/api/rest.go | 26 +++----------------------- backend/app/rest/api/rss.go | 2 +- backend/remark.rest | 3 +++ 3 files changed, 7 insertions(+), 24 deletions(-) diff --git a/backend/app/rest/api/rest.go b/backend/app/rest/api/rest.go index 4ab25249..21223f5c 100644 --- a/backend/app/rest/api/rest.go +++ b/backend/app/rest/api/rest.go @@ -183,7 +183,7 @@ func (s *Rest) routes() chi.Router { authHandler, avatarHandler := s.Authenticator.Handlers() router.Group(func(r chi.Router) { - l := logger.New(logger.Flags(logger.All), logger.IPfn(ipFn)) + l := logger.New(logger.Flags(logger.All), logger.IPfn(ipFn), logger.Prefix("[INFO]")) r.Use(l.Handler, tollbooth_chi.LimitHandler(tollbooth.NewLimiter(5, nil))) r.Mount("/auth", authHandler) }) @@ -195,26 +195,6 @@ func (s *Rest) routes() chi.Router { authMiddleware := s.Authenticator.Middleware() - //// auth routes for all providers - //router.Route("/auth", func(r chi.Router) { - // l := logger.New(logger.Flags(logger.All), logger.IPfn(ipFn)) - // r.Use(l.Handler, tollbooth_chi.LimitHandler(tollbooth.NewLimiter(5, nil))) - // - // for _, provider := range s.Authenticator.Providers { - // r.Mount("/"+provider.Name, provider.Routes()) // mount auth providers as /auth/{name} - // } - // if len(s.Authenticator.Providers) > 0 { - // // shortcut, can be any of providers, all logouts do the same - removes cookie - // r.Get("/logout", s.Authenticator.Providers[0].LogoutHandler) - // } - //}) - - //avatarMiddlewares := []func(http.Handler) http.Handler{ - // logger.New(logger.Flags(logger.None)).Handler, - // tollbooth_chi.LimitHandler(tollbooth.NewLimiter(100, nil)), - //} - //router.Mount(s.AvatarProxy.Routes(avatarMiddlewares...)) // mount avatars to /api/v1/avatar/{file.img} - // api routes router.Route("/api/v1", func(rapi chi.Router) { @@ -227,7 +207,7 @@ func (s *Rest) routes() chi.Router { rapi.Group(func(ropen chi.Router) { ropen.Use(tollbooth_chi.LimitHandler(tollbooth.NewLimiter(10, nil))) ropen.Use(authMiddleware.Trace) - ropen.Use(logger.New(logger.Flags(logger.All), logger.IPfn(ipFn)).Handler) + ropen.Use(logger.New(logger.Flags(logger.All), logger.Prefix("[INFO]"), logger.IPfn(ipFn)).Handler) ropen.Get("/find", s.findCommentsCtrl) ropen.Get("/id/{id}", s.commentByIDCtrl) ropen.Get("/comments", s.findUserCommentsCtrl) @@ -247,7 +227,7 @@ func (s *Rest) routes() chi.Router { rapi.Group(func(rauth chi.Router) { rauth.Use(tollbooth_chi.LimitHandler(tollbooth.NewLimiter(10, nil))) rauth.Use(authMiddleware.Auth) - rauth.Use(logger.New(logger.Flags(logger.All), logger.IPfn(ipFn)).Handler) + rauth.Use(logger.New(logger.Flags(logger.All), logger.Prefix("[INFO]"), logger.IPfn(ipFn)).Handler) rauth.Post("/comment", s.createCommentCtrl) rauth.Put("/comment/{id}", s.updateCommentCtrl) rauth.Get("/user", s.userInfoCtrl) diff --git a/backend/app/rest/api/rss.go b/backend/app/rest/api/rss.go index 98ff4e12..a64b4389 100644 --- a/backend/app/rest/api/rss.go +++ b/backend/app/rest/api/rss.go @@ -113,7 +113,7 @@ func (s *Rest) rssRepliesCtrl(w http.ResponseWriter, r *http.Request) { if len(replies) > maxRssItems || c.Timestamp.Add(maxReplyDuration).Before(time.Now()) { break } - if c.ParentID != "" && !c.Deleted && c.User.ID != userID { // not interested replies to yourself + if c.ParentID != "" && !c.Deleted && c.User.ID != userID { // not interested in replies to yourself var pc store.Comment if pc, e = s.DataService.Get(c.Locator, c.ParentID); e != nil { return nil, errors.Wrap(e, "can't get parent comment") diff --git a/backend/remark.rest b/backend/remark.rest index 6d0e0f97..89cbe847 100644 --- a/backend/remark.rest +++ b/backend/remark.rest @@ -107,6 +107,9 @@ GET {{host}}/api/v1/rss/post?site={{site}}&url={{url}} ### site rss GET {{host}}/api/v1/rss/site?site={{site}} +### replie rss +GET {{host}}/api/v1/rss/reply?site={{site}}&user={{user}} + ### get default avatar GET {{host}}/api/v1/avatar/blah