Whitespace is stripped out of build name in file spec

[darwish]

I have a file spec that looks like this:

{
    "files": [
        {
            "aql": {
                "items.find": {
                    "@build.name": "myproject :: mybranch"
                }
            }
        }
    ]
}

This naming convention for builds comes from the Artifactory Plugin for Jenkins

If I use the cli to do a search, the whitespace seems to get removed from my query:

$ JFROG_CLI_LOG_LEVEL=DEBUG jfrog rt s --spec filespec.json
[Info] Searching artifacts...
[Debug] Searching Artifactory using AQL query:  items.find({"@build.name":"myproject::mybranch"}).include("name","repo","path","actual_md5","actual_sha1","size","property","type")

which obviously doesn't find what I'm looking for.

The issue seems to be at https://github.com/JFrogDev/jfrog-cli-go/blob/c1b328e6667df562f4d774bdbd2b1826dc485a66/artifactory/utils/specutils.go#L52. I can confirm that replacing aql.ItemsFind = cliutils.StripChars(str[first:last], "\n\t ") with aql.ItemsFind = str[first:last] fixes the issue, although it loses all minification entirely:

$ JFROG_CLI_LOG_LEVEL=DEBUG ./jfrog rt s --spec ~/code/testengine/tts/filespec.json
[Info] Searching artifacts...
[Debug] Searching Artifactory using AQL query:  items.find( {
                    "@build.name": "myproject :: mybranch"
                }
            ).include("name","repo","path","actual_md5","actual_sha1","size","property","type")

[eyalbe4]

Thanks for reporting this @darwish.
We'll work on fixing this and keep you posted.

[yahavi]

Thanks, @darwish.
We fixed this issue.
Here is the link to the commit: 3cbce8c

[eyalbe4]

@darwish,
Version 1.10.2 is released and includes this fix.