Merge pull request #256 from devfeel/develop

use strings.Contains instead strings.Index, Thanks for @testwill!
devfeel · Dec 13, 2023 · d50e13e · d50e13e
2 parents a96787b + 5473ef4
commit d50e13e
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/consts.go b/consts.go
@@ -3,7 +3,7 @@ package dotweb
 // Global define
 const (
 	// Version current version
-	Version = "1.7.21"
+	Version = "1.7.22"
 )
 
 // Log define

diff --git a/server.go b/server.go
@@ -533,7 +533,7 @@ type LogJson struct {
 // check request is the websocket request
 // check Connection contains upgrade
 func checkIsWebSocketRequest(req *http.Request) bool {
-	if strings.Index(strings.ToLower(req.Header.Get("Connection")), "upgrade") >= 0 {
+	if strings.Contains(strings.ToLower(req.Header.Get("Connection")), "upgrade") {
 		return true
 	}
 	return false

diff --git a/version.MD b/version.MD
@@ -1,5 +1,10 @@
 ## dotweb版本记录：
 
+####Version 1.7.22
+* tip: use strings.Contains instead strings.Index on server.go
+* Thanks for @testwill #253!
+* 2023-12-13 08:00 at ShangHai
+
 ####Version 1.7.21
 * feature: add SessionManager.RemoveSessionState to delete the session state associated with a specific session ID
 * feature: add HttpContext.DestorySession() to delete all contents of the session and set the sessionId to empty