queueit_knownuserv3 3.6.0 → 3.6.1

data/.gitignore

@@ -0,0 +1,5 @@
+################################################################################
+# This .gitignore file was automatically created by Microsoft(R) Visual Studio.
+################################################################################
+
+/.vs

data/lib/queueit_knownuserv3/user_in_queue_service.rb

@@ -3,7 +3,7 @@ require 'cgi'
 
 module QueueIt
 	class UserInQueueService
-		SDK_VERSION_NO = "3.6.0"
+		SDK_VERSION_NO = "3.6.1"
 		SDK_VERSION = "v3-ruby-" + SDK_VERSION_NO
     
 		def initialize(userInQueueStateRepository)
@@ -27,11 +27,27 @@ module QueueIt
 
 			queueParams = QueueUrlParams::extractQueueParams(queueitToken)
 
-			if(!queueParams.nil?) 
-				return getQueueITTokenValidationResult(targetUrl, config, queueParams, customerId, secretKey)
+			requestValidationResult = nil
+            isTokenValid = false
+
+			if (!queueParams.nil?) 
+				tokenValidationResult = validateToken(config, queueParams, secretKey)
+				isTokenValid = tokenValidationResult.isValid
+
+				if (isTokenValid)
+					requestValidationResult = getValidTokenResult(config, queueParams, secretKey)
+				else
+					requestValidationResult = getErrorResult(customerId, targetUrl, config, queueParams, tokenValidationResult.errorCode)
+				end
 			else 
-				return cancelQueueCookieReturnQueueResult(targetUrl, config, customerId)
+				requestValidationResult = getQueueResult(targetUrl, config, customerId)
+            end
+            
+			if (state.isFound && !isTokenValid)
+				@userInQueueStateRepository.cancelQueueCookie(config.eventId, config.cookieDomain);
 			end
+            
+            return requestValidationResult;
 		end
 
 		def validateCancelRequest(targetUrl, cancelConfig, customerId, secretKey)
@@ -49,18 +65,7 @@ module QueueIt
 			end
 		end 
 
-		def getQueueITTokenValidationResult(targetUrl, config, queueParams,customerId, secretKey) 
-			calculatedHash = OpenSSL::HMAC.hexdigest('sha256', secretKey, queueParams.queueITTokenWithoutHash) 
-			if (calculatedHash.upcase() != queueParams.hashCode.upcase()) 
-				return cancelQueueCookieReturnErrorResult(customerId, targetUrl, config, queueParams, "hash")
-			end
-			if (queueParams.eventId.upcase() != config.eventId.upcase()) 
-				return cancelQueueCookieReturnErrorResult(customerId, targetUrl, config, queueParams, "eventid")
-			end
-			if (queueParams.timeStamp < Time.now.getutc.tv_sec) 
-				return cancelQueueCookieReturnErrorResult(customerId, targetUrl, config, queueParams, "timestamp")
-			end
-
+		def getValidTokenResult(config, queueParams, secretKey) 
 			@userInQueueStateRepository.store(
 				config.eventId,
 				queueParams.queueId,
@@ -68,12 +73,11 @@ module QueueIt
 				!Utils::isNilOrEmpty(config.cookieDomain) ? config.cookieDomain : '',
 				queueParams.redirectType,
 				secretKey)
+
 			return RequestValidationResult.new(ActionTypes::QUEUE, config.eventId, queueParams.queueId, nil, queueParams.redirectType, config.actionName)
 		end
 
-		def cancelQueueCookieReturnErrorResult(customerId, targetUrl, config, qParams, errorCode) 
-			@userInQueueStateRepository.cancelQueueCookie(config.eventId, config.cookieDomain)
-
+		def getErrorResult(customerId, targetUrl, config, qParams, errorCode) 
 			query = getQueryString(customerId, config.eventId, config.version, config.actionName, config.culture, config.layoutName) +
 				"&queueittoken=" + qParams.queueITToken +
 				"&ts=" + Time.now.getutc.tv_sec.to_s +
@@ -84,9 +88,7 @@ module QueueIt
 			return RequestValidationResult.new(ActionTypes::QUEUE, config.eventId, nil, redirectUrl, nil, config.actionName)        
 		end
 
-		def cancelQueueCookieReturnQueueResult(targetUrl, config, customerId) 
-			@userInQueueStateRepository.cancelQueueCookie(config.eventId, config.cookieDomain)
-
+		def getQueueResult(targetUrl, config, customerId) 
 			query = getQueryString(customerId, config.eventId, config.version, config.actionName, config.culture, config.layoutName) + 
 							(!Utils::isNilOrEmpty(targetUrl) ? "&t=" + Utils.urlEncode( targetUrl) : "")		
 
@@ -126,5 +128,30 @@ module QueueIt
 		def getIgnoreActionResult(actionName)
 			return RequestValidationResult.new(ActionTypes::IGNORE, nil, nil, nil, nil, actionName)
 		end
+
+		def validateToken(config, queueParams, secretKey)
+			calculatedHash = OpenSSL::HMAC.hexdigest('sha256', secretKey, queueParams.queueITTokenWithoutHash) 
+			if (calculatedHash.upcase() != queueParams.hashCode.upcase()) 
+				return TokenValidationResult.new(false, "hash")
+			end
+			if (queueParams.eventId.upcase() != config.eventId.upcase()) 
+				return TokenValidationResult.new(false, "eventid")
+			end
+			if (queueParams.timeStamp < Time.now.getutc.tv_sec) 
+				return TokenValidationResult.new(false, "timestamp")
+			end
+
+			return TokenValidationResult.new(true, nil)
+		end
+
+		class TokenValidationResult
+			attr_reader :isValid
+			attr_reader :errorCode
+
+			def initialize(isValid, errorCode)
+				@isValid = isValid
+				@errorCode = errorCode
+			end
+		end
 	end
 end

data/lib/queueit_knownuserv3/user_in_queue_state_cookie_repository.rb

@@ -143,35 +143,42 @@ module QueueIt
 		end
 
 		def getState(eventId, cookieValidityMinutes, secretKey, validateTime) 
-			cookieKey = self.class.getCookieKey(eventId)
-			if (@cookieManager.getCookie(cookieKey).nil?) 
-				return StateInfo.new(false, nil, nil, nil)
-			end
-			cookieNameValueMap = getCookieNameValueMap(@cookieManager.getCookie(cookieKey))
-			if (!isCookieValid(secretKey, cookieNameValueMap, eventId, cookieValidityMinutes, validateTime))
-				return StateInfo.new(false, nil, nil, nil)
-			end
+			begin
+				cookieKey = self.class.getCookieKey(eventId)
+				if (@cookieManager.getCookie(cookieKey).nil?) 
+					return StateInfo.new(false, false, nil, nil, nil)
+				end
+				cookieNameValueMap = getCookieNameValueMap(@cookieManager.getCookie(cookieKey))
+				if (!isCookieValid(secretKey, cookieNameValueMap, eventId, cookieValidityMinutes, validateTime))
+					return StateInfo.new(true, false, nil, nil, nil)
+				end
 
-			fixedCookieValidityMinutes = nil
-			if (cookieNameValueMap.key?("FixedValidityMins")) 
-				fixedCookieValidityMinutes = cookieNameValueMap["FixedValidityMins"].to_i
-			end
+				fixedCookieValidityMinutes = nil
+				if (cookieNameValueMap.key?("FixedValidityMins")) 
+					fixedCookieValidityMinutes = cookieNameValueMap["FixedValidityMins"].to_i
+				end
 
-			return StateInfo.new(
-				true, 
-				cookieNameValueMap["QueueId"], 
-				fixedCookieValidityMinutes,
-				cookieNameValueMap["RedirectType"])
+				return StateInfo.new(
+					true,
+					true, 
+					cookieNameValueMap["QueueId"], 
+					fixedCookieValidityMinutes,
+					cookieNameValueMap["RedirectType"])
+			rescue
+				return StateInfo.new(true, false, nil, nil, nil)
+			end
 		end
 	end
 
 	class StateInfo 
+		attr_reader :isFound
 		attr_reader :isValid
 		attr_reader :queueId
 		attr_reader :fixedCookieValidityMinutes
 		attr_reader :redirectType
 
-		def initialize(isValid, queueId, fixedCookieValidityMinutes, redirectType) 
+		def initialize(isFound, isValid, queueId, fixedCookieValidityMinutes, redirectType) 
+			@isFound = isFound
 			@isValid = isValid
 			@queueId = queueId
 			@fixedCookieValidityMinutes = fixedCookieValidityMinutes

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: queueit_knownuserv3
 version: !ruby/object:Gem::Version
-  version: 3.6.0
+  version: 3.6.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-03-02 00:00:00.000000000 Z
+date: 2020-06-12 00:00:00.000000000 Z
 dependencies: []
 description: 
 email:
@@ -20,9 +20,7 @@ extra_rdoc_files: []
 files:
 - .devcontainer/Dockerfile
 - .devcontainer/devcontainer.json
-- .vs/VSWorkspaceState.json
-- .vs/knownuser-v3-ruby/v16/.suo
-- .vs/slnx.sqlite
+- .gitignore
 - .vscode/launch.json
 - Gemfile
 - Rakefile

data/.vs/VSWorkspaceState.json

@@ -1,9 +0,0 @@
-{
-  "ExpandedNodes": [
-    "",
-    "\\test",
-    "\\test\\queueit_knownuserv3"
-  ],
-  "SelectedNode": "\\test\\test_queueit_knownuserv3.rb",
-  "PreviewInSolutionExplorer": false
-}