Skip to content

Commit 9d65b00

Browse files
committed
fix: Refactor track endpoint to consolidate routes and avoid trailing slash redirects
1 parent 67cd709 commit 9d65b00

File tree

1 file changed

+4
-11
lines changed

1 file changed

+4
-11
lines changed

internal/server/routes.go

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -88,19 +88,12 @@ func (s *Server) RegisterRouter() http.Handler {
8888
api.DELETE("/events/:id", eventHandler.DeleteEvent)
8989
}
9090

91-
// Track endpoint - supports both /track and /api/track for backwards compatibility
92-
track := router.Group("/track")
93-
track.Use(middleware.RequireAuthApiKey(s.userService))
94-
{
95-
track.POST("/", handler.NewEventHandler(s.eventService).CreateEvent)
96-
}
91+
// Track endpoint - direct route to avoid trailing slash redirects
92+
trackEventHandler := handler.NewEventHandler(s.eventService)
93+
router.POST("/track", middleware.RequireAuthApiKey(s.userService), trackEventHandler.CreateEvent)
9794

9895
// Legacy endpoint - keeping for backwards compatibility
99-
apiTrack := router.Group("/api/track")
100-
apiTrack.Use(middleware.RequireAuthApiKey(s.userService))
101-
{
102-
apiTrack.POST("/", handler.NewEventHandler(s.eventService).CreateEvent)
103-
}
96+
router.POST("/api/track", middleware.RequireAuthApiKey(s.userService), trackEventHandler.CreateEvent)
10497

10598
// Dashboard routes
10699
dashboard := router.Group("/dashboard")

0 commit comments

Comments
 (0)