back round 0 view raw

appview: support git clones for repo.git urls #797

merged

opened by

oppi.li 4 weeks ago targeting master from op/pzylsqyxzukx

instead of rewriting, simply accept .git suffixes in the ResolveRepo middleware.

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

options

unified split

Changed files

+3 -6

appview

middleware

middleware.go

state

router.go

appview/middleware/middleware.go

···

       206
       206
        
       	return func(next http.Handler) http.Handler {

     

       207
       207
        
       		return http.HandlerFunc(func(w http.ResponseWriter, req *http.Request) {

     

       208
       208
        
       			repoName := chi.URLParam(req, "repo")

     

       209
       209
       +
       			repoName = strings.TrimSuffix(repoName, ".git")

     

       210
       210
       +
       

     

       209
       211
        
       			id, ok := req.Context().Value("resolvedId").(identity.Identity)

     

       210
       212
        
       			if !ok {

     

       211
       213
        
       				log.Println("malformed middleware")

+1 -6

appview/state/router.go

···

       67
       67
        
       				http.Redirect(w, r, "/"+redirectPath, http.StatusFound)

     

       68
       68
        
       				return

     

       69
       69
        
       			}

     

       70
       70
       +
       

     

       70
       71
        
       		}

     

       71
       72
        
       

     

       72
       73
        
       		standardRouter.ServeHTTP(w, r)

     
···

       82
       83
        
       		r.Get("/", s.Profile)

     

       83
       84
        
       		r.Get("/feed.atom", s.AtomFeedPage)

     

       84
       85
        
       

     

       85
       85
       -
       		// redirect /@handle/repo.git -> /@handle/repo

     

       86
       86
       -
       		r.Get("/{repo}.git", func(w http.ResponseWriter, r *http.Request) {

     

       87
       87
       -
       			nonDotGitPath := strings.TrimSuffix(r.URL.Path, ".git")

     

       88
       88
       -
       			http.Redirect(w, r, nonDotGitPath, http.StatusMovedPermanently)

     

       89
       89
       -
       		})

     

       90
       90
       -
       

     

       91
       86
        
       		r.With(mw.ResolveRepo()).Route("/{repo}", func(r chi.Router) {

     

       92
       87
        
       			r.Use(mw.GoImport())

     

       93
       88
        
       			r.Mount("/", s.RepoRouter(mw))