gitea

Commit Graph

Author	SHA1	Message	Date
zeripath	71e1ebfa60	Instead of using routerCtx just escape the url before routing (#18086 ) (#18098 ) Backport #18086 A consequence of forcibly setting the RoutePath to the escaped url is that the auto routing to endpoints without terminal slashes fails (Causing #18060.) This failure raises the possibility that forcibly setting the RoutePath causes other unexpected behaviors too. Therefore, instead we should simply pre-escape the URL in the process registering handler. Then the request URL will be properly escaped for all the following calls. Fix #17938 Fix #18060 Replace #18062 Replace #17997 Signed-off-by: Andrew Thornton <art27@cantab.net>	3 years ago
Lunny Xiao	1bfb0a24d8	Refactor routers directory (#15800 ) * refactor routers directory * move func used for web and api to common * make corsHandler a function to prohibit side efects * rm unused func Co-authored-by: 6543 <6543@obermui.de>	4 years ago

Author

SHA1

Message

Date

zeripath

71e1ebfa60

Instead of using routerCtx just escape the url before routing (#18086 ) (#18098 )

Backport #18086

A consequence of forcibly setting the RoutePath to the escaped url is that the
auto routing to endpoints without terminal slashes fails (Causing #18060.) This
failure raises the possibility that forcibly setting the RoutePath causes other
unexpected behaviors too.

Therefore, instead we should simply pre-escape the URL in the process registering
handler. Then the request URL will be properly escaped for all the following calls.

Fix #17938
Fix #18060
Replace #18062
Replace #17997

Signed-off-by: Andrew Thornton <art27@cantab.net>

Lunny Xiao

1bfb0a24d8

Refactor routers directory (#15800 )

* refactor routers directory

* move func used for web and api to common

* make corsHandler a function to prohibit side efects

* rm unused func

Co-authored-by: 6543 <6543@obermui.de>

2 Commits (e4120bbc899810f1c98da736ef80fcce11d9ca83)