From 9255fd60070166a8c3b95185cd610184bb05d3a3 Mon Sep 17 00:00:00 2001
From: David Lundgren <dlundgren@syberisle.net>
Date: Fri, 26 Jan 2024 15:15:39 -0600
Subject: [PATCH] #365 Use `fetch` and `lookup('ini')` to get the version

---
 .../jfrog/platform/roles/artifactory/tasks/upgrade.yml | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/Ansible/ansible_collections/jfrog/platform/roles/artifactory/tasks/upgrade.yml b/Ansible/ansible_collections/jfrog/platform/roles/artifactory/tasks/upgrade.yml
index b1531fc0..e16881e0 100644
--- a/Ansible/ansible_collections/jfrog/platform/roles/artifactory/tasks/upgrade.yml
+++ b/Ansible/ansible_collections/jfrog/platform/roles/artifactory/tasks/upgrade.yml
@@ -57,15 +57,15 @@
     state: directory
 
 - name: Check artifactory version
-  ansible.builtin.shell: |
-    set -o pipefail;
-    grep artifactory.product.version "{{ artifactory_home }}/app/artifactory.product.version.properties" |cut -d= -f2
-  register: check_version_cmd
+  ansible.builtin.fetch:
+    src: "{{ artifactory_home }}/app/artifactory.product.version.properties"
+    dest: "/tmp/artifactory.product.version.properties"
+    flat: true
   changed_when: false
 
 - name: Set running_version
   ansible.builtin.set_fact:
-    running_version: "{{ check_version_cmd.stdout }}"
+    running_version: "{{ lookup('ansible.builtin.ini', 'artifactory.product.version', type='properties', file='/tmp/artifactory.product.version.properties') }}"
 
 - name: Delete artifactory app directory
   become: true