GenerateAndExposeOpenapi fails to validate

[djgilcrease]

Following the examples for multi path params or trying the single path param example causes
fails to validate swagger: invalid paths: operation GET /actions/{id:[1-9a-z]+}.{format:yml|yaml|json} must define exactly all path parameters (missing: [id:[1-9a-z]+ format:yml|yaml|json id:[1-9a-z]+.{format:yml|yaml|json}])

func main() {
	r = mux.NewRouter().StrictSlash(true)
	s := r.PathPrefix("/api/v1").Subrouter()
	router, _ := swagger.NewRouter(gorilla.NewRouter(s), swagger.Options{
		Context: ctx,
		Openapi: &openapi3.T{
			Info: &openapi3.Info{
				Title:   "my title",
				Version: "1.0.0",
			},
		},
	})
	router.AddRoute(
		http.MethodGet
		"/actions/{id:[1-9a-z]+}.{format:yml|yaml|json}",
		render,
		swagger.Definitions{
			Querystring: swagger.ParameterValue{
				"id": {
					Schema: &swagger.Schema{Value: ""},
				},
				"format": {
					Schema: &swagger.Schema{Value: ""},
				},
			},
		},
	)
	if err := router.GenerateAndExposeOpenapi(); err != nil {
		panic(err)
	}
	srv := &http.Server{
		Handler: r,
		Addr:    "127.0.0.1:8080",
		// Good practice: enforce timeouts for servers you create!
		WriteTimeout: 15 * time.Second,
		ReadTimeout:  15 * time.Second,
	}
	err := srv.ListenAndServe()
}

To get this path to work I had to change it to

func main() {
	r = mux.NewRouter().StrictSlash(true)
	s := r.PathPrefix("/api/v1").Subrouter()
	router, _ := swagger.NewRouter(gorilla.NewRouter(s), swagger.Options{
		Context: ctx,
		Openapi: &openapi3.T{
			Info: &openapi3.Info{
				Title:   "my title",
				Version: "1.0.0",
			},
		},
	})
	router.AddRoute(
		http.MethodGet
		"/actions/{id:[1-9a-z]+}.{format:yml|yaml|json}",
		render,
		swagger.Definitions{
			PathParams: swagger.ParameterValue{
				"id": {
					Schema: &swagger.Schema{Value: ""},
				},
				"format": {
					Schema: &swagger.Schema{Value: ""},
				},
			},
		},
	)
	if err := router.GenerateAndExposeOpenapi(); err != nil {
		panic(err)
	}
	srv := &http.Server{
		Handler: r,
		Addr:    "127.0.0.1:8080",
		// Good practice: enforce timeouts for servers you create!
		WriteTimeout: 15 * time.Second,
		ReadTimeout:  15 * time.Second,
	}
	err := srv.ListenAndServe()
}

[davidebianchi]

Hi @djgilcrease! The function which autogenerate the path parameter configuration does not allow multiple params in a single segment. We could improve the autogenerate function to support this use cases.

As you proposed, it could be resolved until this update setting manually the path parameters.