Merge branch 'next' into SOARHELP-2947

README.md

@@ -6,7 +6,7 @@ Connector Version: 3.2.0
 Product Vendor: Generic  
 Product Name: SMTP  
 Product Version Supported (regex): ".\*"  
-Minimum Product Version: 6.1.0  
+Minimum Product Version: 6.1.1  
 
 This app provides the ability to send email using SMTP
 

release_notes/3.1.1.md

@@ -0,0 +1 @@
+* Bug fix for accessing vault temp directory path [PAPP-32426]

smtp.json

@@ -11,7 +11,7 @@
     "product_vendor": "Generic",
     "product_name": "SMTP",
     "product_version_regex": ".*",
-    "min_phantom_version": "6.1.0",
+    "min_phantom_version": "6.1.1",
     "rest_handler": "request_handler.handle_request",
     "license": "Copyright (c) 2016-2024 Splunk Inc.",
     "logo": "logo_splunk.svg",
@@ -699,9 +699,6 @@
             }
         }
     ],
-    "pip_dependencies": {
-        "wheel": []
-    },
     "pip39_dependencies": {
         "wheel": [
             {
@@ -722,7 +719,7 @@
             },
             {
                 "module": "soupsieve",
-                "input_file": "wheels/py3/soupsieve-2.4.1-py3-none-any.whl"
+                "input_file": "wheels/py3/soupsieve-2.5-py3-none-any.whl"
             },
             {
                 "module": "tinycss2",
@@ -734,4 +731,4 @@
             }
         ]
     }
-}
+}

smtp_connector.py

@@ -41,7 +41,6 @@
 from bs4 import BeautifulSoup
 from phantom.action_result import ActionResult
 from phantom.base_connector import BaseConnector
-from phantom.vault import Vault
 
 from request_handler import RequestStateHandler, _get_dir_name_from_app_name
 from smtp_consts import *
@@ -707,23 +706,15 @@ def _add_attachments(self, outer, attachments, action_result, message_encoding):
                 if '.pdf' not in attachment_vault_id:
                     return action_result.set_status(phantom.APP_ERROR, SMTP_ERROR_SMTP_SEND_EMAIL)
 
-                if hasattr(Vault, "get_phantom_home"):
-                    report_dir_pre_4_0 = '{0}/www/reports'.format(self.get_phantom_home())
-                    report_dir_post_4_0 = '{0}/vault/reports'.format(self.get_phantom_home())
-                else:
-                    report_dir_pre_4_0 = '/opt/phantom/www/reports'
-                    report_dir_post_4_0 = '/opt/phantom/vault/reports'
+                phantom_home_path = self.get_phantom_home()
+                report_dir = os.path.join(phantom_home_path, "vault", "reports")
 
-                filename = ''
-                for report_dir in (report_dir_post_4_0, report_dir_pre_4_0):
-                    test_filename = os.path.join(report_dir, attachment_vault_id)
-                    test_filename = os.path.abspath(test_filename)
+                test_filename = os.path.join(report_dir, attachment_vault_id)
+                test_filename = os.path.abspath(test_filename)
 
-                    if os.path.isfile(test_filename):
-                        filename = test_filename
-                        break
+                filename = test_filename if os.path.isfile(test_filename) else ""
 
-                is_valid_path = filename.startswith(report_dir_pre_4_0) or filename.startswith(report_dir_post_4_0)
+                is_valid_path = filename.startswith(report_dir)
 
                 if not filename or not is_valid_path:
                     return action_result.set_status(phantom.APP_ERROR, SMTP_ERROR_SMTP_SEND_EMAIL)