Merge PR #1705: baseapp: Allow alphanumerics in routes

Previously only alphabetic characters were allowed.

CHANGELOG.md

@@ -1,6 +1,6 @@
 # Changelog
 
-## TBD
+## PENDING
 
 BREAKING CHANGES
 
@@ -8,6 +8,7 @@ FEATURES
 * [lcd] Can now query governance proposals by ProposalStatus
 
 IMPROVEMENTS
+* [baseapp] Allow any alphanumeric character in route
 
 BUG FIXES
 

baseapp/baseapp_test.go

@@ -290,7 +290,7 @@ func (tx txTest) GetMsgs() []sdk.Msg { return tx.Msgs }
 
 const (
 	typeMsgCounter  = "msgCounter"
-	typeMsgCounter2 = "msgCounterTwo" // NOTE: no numerics (?)
+	typeMsgCounter2 = "msgCounter2"
 )
 
 // ValidateBasic() fails on negative counters.

baseapp/router.go

@@ -31,12 +31,12 @@ func NewRouter() *router {
 	}
 }
 
-var isAlpha = regexp.MustCompile(`^[a-zA-Z]+$`).MatchString
+var isAlphaNumeric = regexp.MustCompile(`^[a-zA-Z0-9]+$`).MatchString
 
 // AddRoute - TODO add description
 func (rtr *router) AddRoute(r string, h sdk.Handler) Router {
-	if !isAlpha(r) {
-		panic("route expressions can only contain alphabet characters")
+	if !isAlphaNumeric(r) {
+		panic("route expressions can only contain alphanumeric characters")
 	}
 	rtr.routes = append(rtr.routes, route{r, h})