Contents of cfif tag removed

gpickin commented 4 years ago

Contents of tag removed in my Application.cfc It didn't seem to do this on surrounding cfif tags, might it be a CASE thing?

It was <CFIF FindNoCase("domain.com",CGI.SERVER_NAME) EQ "0"> It became <CFIF>

Below is the surrounding code incase that influences the behavior somehow.

BEFORE

<cffunction name="onRequest" returnType="void">
        <cfargument name="thePage" type="string" required="true">
        <cfset var tempCall = setupWireboxLite()>

        <!--- <cfif FindNoCase("www.domain.com",CGI.Server_Name) neq "0" and CGI.Server_Port_Secure eq "0">
            <cfif CGI.Script_Name neq "/msc.cfm">
                <cflocation url="https://www.domain.com#CGI.Script_Name#?#CGI.Query_String#&secureRelocate=true" Addtoken="No">
            </cfif>
        </cfif>  --->
        <CFIF FindNoCase("domain.com",CGI.SERVER_NAME) EQ "0">
            <CFLOCATION url="https://domain.com#CGI.SCRIPT_NAME#?#CGI.QUERY_STRING#&secureRelocate=true" Addtoken="No">
        </CFIF>

        <cfset application.right_now = now()>
        <cfinclude template="#arguments.thePage#">
    </cffunction>

AFTER

<cffunction name="onRequest" returnType="void">
        <cfargument name="thePage" type="string" required="true">
        <cfset var tempCall = setupWireboxLite()>

        <!---
            <cfif FindNoCase("www.domain.com",CGI.Server_Name) neq "0" and CGI.Server_Port_Secure eq "0">
            <cfif CGI.Script_Name neq "/msc.cfm">
            <cflocation url="https://www.domain.com#CGI.Script_Name#?#CGI.Query_String#&secureRelocate=true" Addtoken="No">
            </cfif>
            </cfif>
        --->
        <CFIF>
            <CFLOCATION url="https://domain.com#CGI.SCRIPT_NAME#?#CGI.QUERY_STRING#&secureRelocate=true" Addtoken="No">
        </CFIF>

        <cfset application.right_now = now()>
        <cfinclude template="#arguments.thePage#">
    </cffunction>

gpickin commented 4 years ago

I changed the case, and it seemed to not remove it... so it looks like case is the reason.

jcberquist commented 4 years ago

Thanks for the report, this is fixed in v0.14.1

jcberquist / commandbox-cfformat

Contents of cfif tag removed #69