Merge pull request #646 from router-for-me/amp

fix(amp): add /settings routes to proxy

Luis Pater committed 10 months ago

Commit fcd64752 parents 453e744 3fc410a1 file · +3 −1Browse files

Unified · Split

internal/api/modules/amp/routes.go+3−1

@@ -126,7 +126,7 @@ func (m AmpModule) registerManagementRoutes(engine gin.Engine, baseHandler *ha · 125 unmodified lines
126	126	var authWithBypass gin.HandlerFunc
127	127	if auth != nil {
128	128	ampAPI.Use(auth)
129		− authWithBypass = wrapManagementAuth(auth, "/threads", "/auth", "/docs")
	129	+ authWithBypass = wrapManagementAuth(auth, "/threads", "/auth", "/docs", "/settings")
130	130	}
131	131
132	132	// Dynamic proxy handler that uses m.getProxy() for hot-reload support
@@ -179,6 +179,8 @@ func (m AmpModule) registerManagementRoutes(engine gin.Engine, baseHandler *ha · 46 unmodified lines
179	179	engine.GET("/threads/*path", append(rootMiddleware, proxyHandler)...)
180	180	engine.GET("/docs", append(rootMiddleware, proxyHandler)...)
181	181	engine.GET("/docs/*path", append(rootMiddleware, proxyHandler)...)
	182	+ engine.GET("/settings", append(rootMiddleware, proxyHandler)...)
	183	+ engine.GET("/settings/*path", append(rootMiddleware, proxyHandler)...)
182	184
183	185	engine.GET("/threads.rss", append(rootMiddleware, proxyHandler)...)
184	186	engine.GET("/news.rss", append(rootMiddleware, proxyHandler)...)