184 bugfix

v1/Application.cfc

@@ -1,5 +1,10 @@
-`<cfcomponent extends="taffy.core.api">
+<cfcomponent extends="taffy.core.api">
 	<!---<cfset this.name = "deck_api" />--->
+	<!--- Способы передачи аргументов из Application.cfc
+	В скоупе request (доступен всегда и отовсюду). Но там может возникнуть хаос. Плюс: достаточно явный способ
+	onTaffyRequest.arguments.requestArguments - инжектируется в аргументы методов get/post/put etc/
+	this - в Taffy не работает, a только при include шаблона в Application.cfc (есть такой способ условного вызова шаблона) --->
+	
 	<cfset this.mappings["/resources"] = expandPath("./resources")/>
 	<cfset this.mappings["/taffy"] = expandPath("./taffy")/>
 	<cfset this.mappings["/lib"] = expandPath("./lib")/>	
@@ -52,7 +57,7 @@
 		
 		//variables.framework.docs={};
 		variables.framework.docs.APIName="svc-api";
-		variables.framework.docs.APIVersion="0.183";   /*$ git config --global --unset user.password*/
+		variables.framework.docs.APIVersion="0.184";   /*$ git config --global --unset user.password*/
 		
 		variables.framework.globalHeaders = structNew();
 		variables.framework.globalHeaders["Access-Control-Expose-Headers"] = "Location"; 
@@ -162,13 +167,13 @@
 		
 			<cfswitch expression=#stand#>
 				<cfcase value=",dev">
-					<cfreturn "https://auth-api-dev.ngcloud.ru/api/v1/auth/user"/>
+					<cfreturn "https://auth-api-dev.ngcloud.ru/api/v1" />
 				</cfcase>
 				<cfcase value="test">
-					<cfreturn "https://auth-api-test.ngcloud.ru/api/v1/auth/user"/>
+					<cfreturn "https://auth-api-test.ngcloud.ru/api/v1" />
 				</cfcase>			
 				<cfcase value="prod">
-					<cfreturn "https://auth-api.ngcloud.ru/api/v1/auth/user"/>
+					<cfreturn "https://auth-api.ngcloud.ru/api/v1" />
 				</cfcase>			
 				<cfdefaultcase><!--- proceed to the legacy locator ---></cfdefaultcase>
 			</cfswitch> 
@@ -179,7 +184,7 @@
 		</cftry>
 		
 		<!--- вот если ничего не прописано, попробуем по-старинке --->
-		<cfloop list="https://auth-api-dev.ngcloud.ru/api/v1/auth/user,https://auth-api-test.ngcloud.ru/api/v1/auth/user,https://auth-api.ngcloud.ru/api/v1/auth/user" index="serviceUrl"><!--- *** начиная с девелоперского --->
+		<cfloop list="https://auth-api-dev.ngcloud.ru/api/v1,https://auth-api-test.ngcloud.ru/api/v1,https://auth-api.ngcloud.ru/api/v1" index="serviceUrl"><!--- *** начиная с девелоперского --->
 			<cftry>	
 				<cfhttp url=#serviceUrl# method="OPTIONS" result="resp" timeout=1/>
 				
@@ -251,19 +256,20 @@
 		}
 		var result="";
 		try {
-			var httpService = new http(method = "GET", charset = "utf-8", url = #this.iamServiceUrl#, timeout="5");
+			var authUrl =  "#this.iamServiceUrl#/auth/user";
+			var httpService = new http(method = "GET", charset = "utf-8", url = #authUrl#, timeout="5");
 			httpService.addParam(type = "HEADER", name = "Accept",  value = "application/json");
 			httpService.addParam(type = "HEADER", name = "Authorization",  value = "#request.auth_header#"); //passthrough	
-			//writedump(this.iamServiceUrl);abort;
+			//writedump(authUrl);abort;
 			var resp = httpService.send();	
 			//if (resp.status_code NEQ 200) throw("IDP response not OK");			
 			//writedump(resp);abort;
 			var prefix = resp.getPrefix();
 			if (prefix.status_code NEQ 200) {
-				//writedump(this.iamServiceUrl);
+				//writedump(authUrl);
 				//writedump(resp);
 				var iamStatusCode = (isValid("integer", prefix.status_code)) ? val(prefix.status_code) : 500;
-				return representationOf( {"iamServiceUrl"=this.iamServiceUrl, "idpResponse"=resp} ).withStatus(iamStatusCode);
+				return representationOf( {"IAM URL"=#authUrl#, "idpResponse"=resp} ).withStatus(iamStatusCode);
 				//abort;				
 				//throw("IDP response not OK");
 			}

v1/resources/vault_record.cfc

@@ -21,6 +21,19 @@
 		<cfset var local={}/>
 		<cfset local.vaultData={}/>
 		
+		<!--- <cfset logVaultAccess(
+			"#request.iam_service_url#/events",
+			request.auth_header,
+			request.stand,
+			arguments.instanceUid,
+			arguments.secretName,
+			"",			
+			arguments.login,
+			arguments.clientID,
+			arguments.usrId,
+			arguments.contragentId,
+			arguments.isImpersonated)/> --->
+		
 		<!--- *** Добавить проверку, что секрет перечислен в fields - стало быть, доступен пользователю --->
 		
 		<!--- тут необходимо проконтролировать, что мы смотрим не чужие данные --->
@@ -57,13 +70,17 @@
 		
 		<!--- <cfdump var=#local.qCurrentState#/><cfabort/> --->
 		<cfset var vaultUrl=""/>
-		<cftry>		
+		<cftry>					
-			
 			<cfset var instanceData=#deserializeJson(local.qCurrentState.instance_data)#/>
 			<cfif structKeyExists(instanceData,"vaultUrl")>
 				<cfset vaultUrl=instanceData.vaultUrl/>	<!--- v001 --->
 			<cfelseif structKeyExists(instanceData,"vault")>
+				<cfif !structKeyExists(instanceData.vault,"Url")>
+					<cfreturn representationOf("instanceData.vault does not contain url").withStatus(422)/>
+				</cfif>
 				<cfset vaultUrl=instanceData.vault.Url/><!--- v002 --->	
+			<cfelse>
+				<cfreturn representationOf("Instance data do not contain vault url").withStatus(422)/>
 			</cfif>
 			
 			<cfhttp method="post" 
@@ -122,7 +139,7 @@
 		
 		<!--- логируем успешную попытку доступа. (если логировать сначала - неизвестен результат. *** Можно в catch логировать неуспешный --->
 		<cfset logVaultAccess(
-			request.iam_service_url,
+			"#request.iam_service_url#/events",
 			request.auth_header,
 			request.stand,
 			arguments.instanceUid,
@@ -143,7 +160,7 @@
 	</cffunction>
 
 	<cffunction name="logVaultAccess" returntype="void">
-		<cfargument name="iamUrl" type="string" required=true hint=""/>
+		<cfargument name="loggerUrl" type="string" required=true hint=""/>
 		<cfargument name="authHeader" type="string" required=true hint="type:string"/>
 		<cfargument name="stand" type="string" required=true hint="type:string"/>
 		<cfargument name="instanceUid" type="string" required=true hint="type:UUID"/>
@@ -194,7 +211,7 @@
 			<!--- <cfdump var=#msg#/><cfabort/> --->
 			
 			<cfhttp method="post" 
-				url="#arguments.iamUrl#" 
+				url="#arguments.loggerUrl#" 
 				timeout="3"
 				charset="utf-8"	
 				result="local.res"
@@ -204,7 +221,12 @@
 				<cfhttpparam type="BODY" value="#serializeJson(msg)#"/>					
 			</cfhttp>
 		
-			<!--- <cfdump var=#local.res#/><cfabort/> --->
+			<!---  
+			arguments:<cfdump var=#arguments#/>
+			msg:<cfdump var=#msg#/>
+			local.res:<cfdump var=#local.res#/>
+			<cfabort/>
+			--->
 			<cfcatch type="any">
 				<cfrethrow/><!--- на этапе отладки лучше тут падать --->
 			</cfcatch>