From e868f13b6713be89733490b4eb9da4504c283ec7 Mon Sep 17 00:00:00 2001 From: Michael Yochpaz <8832013+MichaelYochpaz@users.noreply.github.com> Date: Thu, 29 Feb 2024 11:26:12 +0200 Subject: [PATCH 1/7] Update Docker version --- .../Scripts/ConvertFile/ConvertFile.py | 25 +++++++++---------- .../Scripts/ConvertFile/ConvertFile.yml | 2 +- 2 files changed, 13 insertions(+), 14 deletions(-) diff --git a/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile.py b/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile.py index 0706fe36a9a9..07fbdc292278 100644 --- a/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile.py +++ b/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile.py @@ -7,7 +7,6 @@ import tempfile import shutil import traceback -from typing import List def find_zombie_processes(): @@ -29,35 +28,35 @@ def find_zombie_processes(): return zombies, ps_out -def convert_file(file_path: str, out_format: str, all_files: bool, outdir: str) -> List[str]: +def convert_file(file_path: str, out_format: str, all_files: bool, outdir: str) -> list[str]: try: run_cmd = ['soffice', '--headless', '-env:UserInstallation=file:///tmp/convertfile/.config', '--convert-to', out_format, file_path, '--outdir', outdir] env = os.environ.copy() env['HOME'] = '/tmp/convertfile' res = subprocess.check_output(run_cmd, stderr=subprocess.STDOUT, universal_newlines=True, env=env) - demisto.debug("completed running: {}. With result: {}".format(run_cmd, res)) + demisto.debug(f"completed running: {run_cmd}. With result: {res}") if all_files: files = glob.glob(outdir + '/*') else: ext = out_format.split(':')[0] files = glob.glob(outdir + '/*.' + ext) if not files: - raise ValueError('Failed convert for output format: {}. Convert process log: {}'.format(out_format, res)) + raise ValueError(f'Failed convert for output format: {out_format}. Convert process log: {res}') return files finally: # make sure we don't have zombie processes (seen when converting pdf to html) try: zombies, ps_out = find_zombie_processes() if zombies: # pragma no cover - demisto.info("Found zombie processes will waitpid: {}".format(ps_out)) + demisto.info(f"Found zombie processes will waitpid: {ps_out}") for pid in zombies: waitres = os.waitpid(int(pid), os.WNOHANG) - demisto.info("waitpid result: {}".format(waitres)) + demisto.info(f"waitpid result: {waitres}") else: - demisto.debug("No zombie processes found for ps output: {}".format(ps_out)) + demisto.debug(f"No zombie processes found for ps output: {ps_out}") except Exception as ex: - demisto.error("Failed checking for zombie processes: {}. Trace: {}".format(ex, traceback.format_exc())) + demisto.error(f"Failed checking for zombie processes: {ex}. Trace: {traceback.format_exc()}") def main(): @@ -68,8 +67,8 @@ def main(): try: result = demisto.getFilePath(entry_id) if not result: - return_error("Couldn't find entry id: {}".format(entry_id)) - demisto.debug('going to convert: {}'.format(result)) + return_error(f"Couldn't find entry id: {entry_id}") + demisto.debug(f'going to convert: {result}') file_path = result['path'] file_path_name_only = os.path.splitext(os.path.basename(file_path))[0] file_name = result.get('name') @@ -78,7 +77,7 @@ def main(): with tempfile.TemporaryDirectory() as outdir: files = convert_file(file_path, out_format, all_files, outdir) if not files: - return_error('No file result returned for convert format: {}'.format(out_format)) + return_error(f'No file result returned for convert format: {out_format}') return for f in files: temp = demisto.uniqueFile() @@ -94,9 +93,9 @@ def main(): 'FileID': temp }) except subprocess.CalledProcessError as e: - return_error("Failed converting file. Output: {}. Error: {}".format(e.output, e)) + return_error(f"Failed converting file. Output: {e.output}. Error: {e}") except Exception as e: - return_error("Failed converting file. General exception: {}.\n\nTrace:\n{}".format(e, traceback.format_exc())) + return_error(f"Failed converting file. General exception: {e}.\n\nTrace:\n{traceback.format_exc()}") # python2 uses __builtin__ python3 uses builtins diff --git a/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile.yml b/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile.yml index 163e094272d9..4f664d422f35 100644 --- a/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile.yml +++ b/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile.yml @@ -42,7 +42,7 @@ outputs: description: The file type. type: String scripttarget: 0 -dockerimage: demisto/office-utils:2.0.0.82639 +dockerimage: demisto/office-utils:2.0.0.88457 runas: DBotWeakRole tests: - No tests (auto formatted) From b07867354d55bf0e0024c197d50a4e793f6475c0 Mon Sep 17 00:00:00 2001 From: Michael Yochpaz <8832013+MichaelYochpaz@users.noreply.github.com> Date: Thu, 29 Feb 2024 11:27:09 +0200 Subject: [PATCH 2/7] Bump version --- Packs/CommonScripts/ReleaseNotes/1_14_3.md | 6 ++++++ Packs/CommonScripts/pack_metadata.json | 2 +- 2 files changed, 7 insertions(+), 1 deletion(-) create mode 100644 Packs/CommonScripts/ReleaseNotes/1_14_3.md diff --git a/Packs/CommonScripts/ReleaseNotes/1_14_3.md b/Packs/CommonScripts/ReleaseNotes/1_14_3.md new file mode 100644 index 000000000000..0cd2de00aa5b --- /dev/null +++ b/Packs/CommonScripts/ReleaseNotes/1_14_3.md @@ -0,0 +1,6 @@ + +#### Scripts + +##### ConvertFile + +- Updated the Docker image to: *demisto/office-utils:2.0.0.88457*. diff --git a/Packs/CommonScripts/pack_metadata.json b/Packs/CommonScripts/pack_metadata.json index 49bd808bde8f..129c3f1a3cc2 100644 --- a/Packs/CommonScripts/pack_metadata.json +++ b/Packs/CommonScripts/pack_metadata.json @@ -2,7 +2,7 @@ "name": "Common Scripts", "description": "Frequently used scripts pack.", "support": "xsoar", - "currentVersion": "1.14.2", + "currentVersion": "1.14.3", "author": "Cortex XSOAR", "url": "https://www.paloaltonetworks.com/cortex", "email": "", From 8f29dd2938656194c697cd0134be5a20f37970cc Mon Sep 17 00:00:00 2001 From: Michael Yochpaz <8832013+MichaelYochpaz@users.noreply.github.com> Date: Thu, 29 Feb 2024 11:29:07 +0200 Subject: [PATCH 3/7] ruff --- Packs/CommonScripts/Scripts/ConvertFile/ConvertFile_test.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile_test.py b/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile_test.py index 8ea0a7d8945a..a7fdfa1ca89c 100644 --- a/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile_test.py +++ b/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile_test.py @@ -53,9 +53,9 @@ def test_convert_to_html(mocker): assert results[0]['Type'] == entryTypes['file'] assert results[0]['File'] == 'test.html' glob_list = glob.glob('./*' + results[0]['FileID']) - logging.getLogger().info('glob list for results: {}. list: {}'.format(results[0], glob_list)) + logging.getLogger().info(f'glob list for results: {results[0]}. list: {glob_list}') assert glob_list - with open(glob_list[0], "r") as f: + with open(glob_list[0]) as f: contents = f.read() assert 'Extensions to the Office Open XML' in contents # assert the next result is an image @@ -79,7 +79,7 @@ def test_convert_pdf_to_html(mocker): assert results[0]['Type'] == entryTypes['file'] assert results[0]['File'] == 'test.html' glob_list = glob.glob('./*' + results[0]['FileID']) - logging.getLogger().info('glob list for results: {}. list: {}'.format(results[0], glob_list)) + logging.getLogger().info(f'glob list for results: {results[0]}. list: {glob_list}') assert glob_list # check no defunct processed zombies, output = find_zombie_processes() From 724380e637d46db1ad0fbeccb5ca5932a3b53771 Mon Sep 17 00:00:00 2001 From: Content Bot Date: Thu, 29 Feb 2024 14:33:07 +0000 Subject: [PATCH 4/7] Bump pack from version CommonScripts to 1.14.4. --- Packs/CommonScripts/ReleaseNotes/1_14_4.md | 6 ++++++ Packs/CommonScripts/pack_metadata.json | 2 +- 2 files changed, 7 insertions(+), 1 deletion(-) create mode 100644 Packs/CommonScripts/ReleaseNotes/1_14_4.md diff --git a/Packs/CommonScripts/ReleaseNotes/1_14_4.md b/Packs/CommonScripts/ReleaseNotes/1_14_4.md new file mode 100644 index 000000000000..0cd2de00aa5b --- /dev/null +++ b/Packs/CommonScripts/ReleaseNotes/1_14_4.md @@ -0,0 +1,6 @@ + +#### Scripts + +##### ConvertFile + +- Updated the Docker image to: *demisto/office-utils:2.0.0.88457*. diff --git a/Packs/CommonScripts/pack_metadata.json b/Packs/CommonScripts/pack_metadata.json index 129c3f1a3cc2..a59bd55690e1 100644 --- a/Packs/CommonScripts/pack_metadata.json +++ b/Packs/CommonScripts/pack_metadata.json @@ -2,7 +2,7 @@ "name": "Common Scripts", "description": "Frequently used scripts pack.", "support": "xsoar", - "currentVersion": "1.14.3", + "currentVersion": "1.14.4", "author": "Cortex XSOAR", "url": "https://www.paloaltonetworks.com/cortex", "email": "", From 78114e08a7797f81cda0655f0816bfc062d60aed Mon Sep 17 00:00:00 2001 From: Michael Yochpaz <8832013+MichaelYochpaz@users.noreply.github.com> Date: Sun, 3 Mar 2024 12:04:10 +0200 Subject: [PATCH 5/7] Bump Docker version --- Packs/CommonScripts/ReleaseNotes/1_14_4.md | 2 +- Packs/CommonScripts/Scripts/ConvertFile/ConvertFile.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Packs/CommonScripts/ReleaseNotes/1_14_4.md b/Packs/CommonScripts/ReleaseNotes/1_14_4.md index 0cd2de00aa5b..0673a0e390d2 100644 --- a/Packs/CommonScripts/ReleaseNotes/1_14_4.md +++ b/Packs/CommonScripts/ReleaseNotes/1_14_4.md @@ -3,4 +3,4 @@ ##### ConvertFile -- Updated the Docker image to: *demisto/office-utils:2.0.0.88457*. +- Updated the Docker image to: *demisto/office-utils:2.0.0.88735*. diff --git a/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile.yml b/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile.yml index 4f664d422f35..e4f15dc9ee3f 100644 --- a/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile.yml +++ b/Packs/CommonScripts/Scripts/ConvertFile/ConvertFile.yml @@ -42,7 +42,7 @@ outputs: description: The file type. type: String scripttarget: 0 -dockerimage: demisto/office-utils:2.0.0.88457 +dockerimage: demisto/office-utils:2.0.0.88735 runas: DBotWeakRole tests: - No tests (auto formatted) From ca3ab9c746fbd841d492479dcb0ae92674fe8f5c Mon Sep 17 00:00:00 2001 From: Content Bot Date: Sun, 3 Mar 2024 10:32:35 +0000 Subject: [PATCH 6/7] Bump pack from version CommonScripts to 1.14.5. --- Packs/CommonScripts/ReleaseNotes/1_14_5.md | 6 ++++++ Packs/CommonScripts/pack_metadata.json | 2 +- 2 files changed, 7 insertions(+), 1 deletion(-) create mode 100644 Packs/CommonScripts/ReleaseNotes/1_14_5.md diff --git a/Packs/CommonScripts/ReleaseNotes/1_14_5.md b/Packs/CommonScripts/ReleaseNotes/1_14_5.md new file mode 100644 index 000000000000..0673a0e390d2 --- /dev/null +++ b/Packs/CommonScripts/ReleaseNotes/1_14_5.md @@ -0,0 +1,6 @@ + +#### Scripts + +##### ConvertFile + +- Updated the Docker image to: *demisto/office-utils:2.0.0.88735*. diff --git a/Packs/CommonScripts/pack_metadata.json b/Packs/CommonScripts/pack_metadata.json index a59bd55690e1..4b627d0d2cf7 100644 --- a/Packs/CommonScripts/pack_metadata.json +++ b/Packs/CommonScripts/pack_metadata.json @@ -2,7 +2,7 @@ "name": "Common Scripts", "description": "Frequently used scripts pack.", "support": "xsoar", - "currentVersion": "1.14.4", + "currentVersion": "1.14.5", "author": "Cortex XSOAR", "url": "https://www.paloaltonetworks.com/cortex", "email": "", From c1e932bc35d362252e4f18e0aeee3f2d29ad2d4d Mon Sep 17 00:00:00 2001 From: Content Bot Date: Sun, 3 Mar 2024 13:50:52 +0000 Subject: [PATCH 7/7] Bump pack from version CommonScripts to 1.14.6. --- Packs/CommonScripts/ReleaseNotes/1_14_6.md | 6 ++++++ Packs/CommonScripts/pack_metadata.json | 2 +- 2 files changed, 7 insertions(+), 1 deletion(-) create mode 100644 Packs/CommonScripts/ReleaseNotes/1_14_6.md diff --git a/Packs/CommonScripts/ReleaseNotes/1_14_6.md b/Packs/CommonScripts/ReleaseNotes/1_14_6.md new file mode 100644 index 000000000000..0673a0e390d2 --- /dev/null +++ b/Packs/CommonScripts/ReleaseNotes/1_14_6.md @@ -0,0 +1,6 @@ + +#### Scripts + +##### ConvertFile + +- Updated the Docker image to: *demisto/office-utils:2.0.0.88735*. diff --git a/Packs/CommonScripts/pack_metadata.json b/Packs/CommonScripts/pack_metadata.json index 4b627d0d2cf7..057abdaa5fe8 100644 --- a/Packs/CommonScripts/pack_metadata.json +++ b/Packs/CommonScripts/pack_metadata.json @@ -2,7 +2,7 @@ "name": "Common Scripts", "description": "Frequently used scripts pack.", "support": "xsoar", - "currentVersion": "1.14.5", + "currentVersion": "1.14.6", "author": "Cortex XSOAR", "url": "https://www.paloaltonetworks.com/cortex", "email": "",