Skip to content

Commit e3ef5ac

Browse files
committed
rename trimMultipleSlashes to sanitizeURI
1 parent 5c4cb06 commit e3ef5ac

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

middleware/slash.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ func AddTrailingSlashWithConfig(config TrailingSlashConfig) echo.MiddlewareFunc
6060

6161
// Redirect
6262
if config.RedirectCode != 0 {
63-
return c.Redirect(config.RedirectCode, sanitizeUri(uri))
63+
return c.Redirect(config.RedirectCode, sanitizeURI(uri))
6464
}
6565

6666
// Forward
@@ -108,7 +108,7 @@ func RemoveTrailingSlashWithConfig(config TrailingSlashConfig) echo.MiddlewareFu
108108

109109
// Redirect
110110
if config.RedirectCode != 0 {
111-
return c.Redirect(config.RedirectCode, sanitizeUri(uri))
111+
return c.Redirect(config.RedirectCode, sanitizeURI(uri))
112112
}
113113

114114
// Forward
@@ -120,7 +120,7 @@ func RemoveTrailingSlashWithConfig(config TrailingSlashConfig) echo.MiddlewareFu
120120
}
121121
}
122122

123-
func sanitizeUri(uri string) string {
123+
func sanitizeURI(uri string) string {
124124
// double slash `\\`, `//` or even `\/` is absolute uri for browsers and by redirecting request to that uri
125125
// we are vulnerable to open redirect attack. so replace all slashes from the beginning with single slash
126126
if len(uri) > 1 && (uri[0] == '\\' || uri[0] == '/') && (uri[1] == '\\' || uri[1] == '/') {

0 commit comments

Comments
 (0)