Merge pull request #49 from sashamelentyev/feat/redirect-handler

feat: add check status code in http.RedirectHandler

Author: sashamelentyev

pkg/analyzer/analyzer.go

@@ -116,6 +116,15 @@ func run(pass *analysis.Pass) (interface{}, error) {
 					if basicLit := getBasicLitFromArgs(n.Args, 4, 3, token.INT); basicLit != nil {
 						checkHTTPStatusCode(pass, basicLit)
 					}
+
+				case "RedirectHandler":
+					if !lookupFlag(pass, HTTPStatusCodeFlag) {
+						return
+					}
+
+					if basicLit := getBasicLitFromArgs(n.Args, 2, 1, token.INT); basicLit != nil {
+						checkHTTPStatusCode(pass, basicLit)
+					}
 				}
 			default:
 				if selectorExpr.Sel.Name == "WriteHeader" {

pkg/analyzer/internal/template/test-httpstatuscode.go.tmpl

@@ -114,3 +114,15 @@ func _() {
     http.Redirect(w, r, "", {{ $value }})
 }
 {{ end -}}
+
+{{ range $key, $value := .Mapping }}
+func _() {
+    http.RedirectHandler("", {{ $key }}) // want `"{{ quoteMeta $key }}" can be replaced by {{ quoteMeta $value }}`
+}
+{{ end -}}
+
+{{ range $key, $value := .Mapping }}
+func _() {
+    http.RedirectHandler("", {{ $value }})
+}
+{{ end -}}