Wrong method with trailing slash

[Yopes]

Hello,
I'm encountering an issue with the iris's routes matcher.
I'm using iris v10.2.1
Let's consider the following main:

package main                                                                                                                                                                                                        
                                                                                                                                                                                                                    
import "github.com/kataras/iris"                                                                                                                                                                                    
                                                                                                                                                                                                                    
                                                                                                                                                                                                                    
func main() {                                                                                                                                                                                                       
     app := iris.New()                                                                                                                                                                                              
     app.Handle("GET", "/foo", handler)                                                                                                                                                                          
     app.Handle("POST", "/foo", handler)                                                                                                                                                                         
     app.Handle("GET", "/foo/{bar}", handler)                                                                                                                                                                   
     app.Handle("POST", "/foo/{bar}", handler)                                                                                                                                                                  
     app.Run(iris.Addr(":8082"))                                                                                                                                                                                    
}                                                                                                                                                                                                                   
                                                                                                                                                                                                                    
func handler(ctx iris.Context){                                                                                                                                                                                     
    ctx.Writef("Hello from method: %s and path: %s", ctx.Method(), ctx.Path())                                                                                                                                      
}  

I ran this code and I tried it.

When I do a POST request on http://127.0.0.1:8082/foo => it's working
When I do a GET request on http://127.0.0.1:8082/foo => it's working
When I do a GET request on http://127.0.0.1:8082/foo/ => it's working

But when I do a POST request on http://127.0.0.1:8082/foo/ ,
The result is: "Hello from method: GET and path: /foo"

I guess the ctx.Method() is not supposed to return GET

Thanks for looking at this issue.

[kataras]

Hello @Yopes, yes this is caused because of this: https://github.com/kataras/iris/blob/master/core/router/handler.go#L175 , it redirects with StatusMovedPermantly (301) to let client to cache the redirect in order to gain performance on the next requests with trailing slash.

When automatically redirecting a POST request after receiving a 301 status code, some existing HTTP/1.0 user agents will erroneously change it into a GET request. (that is hapenning here)

The StatusMovedPermantly redirects to GET methods, it doesn't permit POST redirects for security reasons: imagine a shop card, you can't just permantly redirect to one address with the data follows them. For that reason I fixed it with 307 temp redirect status code for both POST and PUT methods.

This was an easy to fix bug but it was important, I always recommend using the /path pattern instead of /path/ but you never know the client's behavior, you did well reporting this behavior, excellent job Kevin!