bump 5.0.0 and fixes for playbooks

Author: mlodic

Diff for: .github/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## [5.0.0](https://github.com/intelowlproject/pyintelowl/releases/tag/5.0.0)
+- Fixes for Playbook Analysis
+
 ## [4.4.7](https://github.com/intelowlproject/pyintelowl/releases/tag/4.4.7)
 - Fixed Running Playbook without TLP set
 

Diff for: docs/conf.py

@@ -13,7 +13,7 @@
 import os
 import sys
 
-VERSION = "4.4.7"
+VERSION = "5.0.0"
 GITHUB_URL = "https://github.com/intelowlproject/pyintelowl"
 
 sys.path.append(os.path.abspath("../"))

Diff for: pyintelowl/cli/_jobs_utils.py

@@ -59,7 +59,7 @@ def _render_job_reports_table(rows, title: str, verbose=False):
         ]
         if verbose:
             for field in ["report", "errors", "runtime_configuration"]:
-                cols.append(get_json_syntax(el[field]) if el[field] else None)
+                cols.append(get_json_syntax(el[field]) if el.get(field, "") else None)
         table.add_row(*cols)
     return table
 

Diff for: pyintelowl/cli/analyse.py

@@ -80,19 +80,8 @@
 # doing it twice to remove --analyzers-list and --connectors-list
 __playbook_analyse_options.pop(0)
 __playbook_analyse_options.pop(0)
-
-__playbook_analyse_options.append(
-    click.option(
-        "-pl",
-        "--playbooks-list",
-        type=str,
-        default="",
-        help="""
-        Comma separated list of playbook names to invoke.
-        Defaults to all configured playbooks.
-        """,
-    ),
-)
+__playbook_analyse_options.pop(3)
+__playbook_analyse_options.pop(2)
 
 
 @click.group("analyse")
@@ -184,57 +173,53 @@ def file(
 
 @analyse.command(help="Send playbook analysis request for an observable")
 @click.argument("value")
+@click.argument("playbook")
 @add_options(__playbook_analyse_options)
 @click.pass_context
 def playbook_observable(
     ctx: ClickContext,
     value: str,
-    playbooks_list: str,
+    playbook: str,
     tags_list: str,
     tlp: str,
-    check,
-    check_minutes_ago: int,
     runtime_config,
     should_poll: bool,
 ):
-    playbooks_list = playbooks_list.split(",") if len(playbooks_list) else []
     tags_labels = tags_list.split(",") if len(tags_list) else []
     if runtime_config:
         runtime_config = get_json_data(runtime_config)
     else:
         runtime_config = {}
     try:
+        print("here")
         ctx.obj._new_analysis_playbook_cli(
             value,
             "observable",
-            check,
+            playbook,
             tlp,
-            playbooks_list,
             runtime_config,
             tags_labels,
             should_poll,
-            check_minutes_ago,
         )
+        print("here3")
     except IntelOwlClientException as e:
         ctx.obj.logger.fatal(str(e))
 
 
 @analyse.command(help="Send playbook analysis request for an observable")
 @click.argument("filepath", type=click.Path(exists=True, resolve_path=True))
+@click.argument("playbook")
 @add_options(__playbook_analyse_options)
 @click.pass_context
 def playbook_file(
     ctx: ClickContext,
     filepath: str,
-    playbooks_list: str,
+    playbook: str,
     tags_list: str,
     tlp: str,
-    check,
-    check_minutes_ago: int,
     runtime_config,
     should_poll: bool,
 ):
-    playbooks_list = playbooks_list.split(",") if len(playbooks_list) else []
     tags_labels = tags_list.split(",") if len(tags_list) else []
     if runtime_config:
         runtime_config = get_json_data(runtime_config)
@@ -244,13 +229,11 @@ def playbook_file(
         ctx.obj._new_analysis_playbook_cli(
             filepath,
             "file",
-            check,
+            playbook,
             tlp,
-            playbooks_list,
             runtime_config,
             tags_labels,
             should_poll,
-            check_minutes_ago,
         )
     except IntelOwlClientException as e:
         ctx.obj.logger.fatal(str(e))

Diff for: pyintelowl/pyintelowl.py

@@ -769,13 +769,11 @@ def _new_analysis_playbook_cli(
         self,
         obj: str,
         type_: str,
-        check,
         playbook: str,
         tlp: TLPType = None,
         runtime_configuration: Dict = None,
         tags_labels: List[str] = None,
         should_poll: bool = False,
-        minutes_ago: int = None,
     ) -> None:
         """
         For internal use by the pyintelowl CLI.
@@ -785,8 +783,6 @@ def _new_analysis_playbook_cli(
         if not tags_labels:
             tags_labels = []
 
-            return
-
         self.logger.info(
             f"""Requesting analysis..
             {type_}: [blue]{obj}[/]
@@ -799,8 +795,8 @@ def _new_analysis_playbook_cli(
         if type_ == "observable":
             resp = self.send_observable_analysis_playbook_request(
                 observable_name=obj,
-                tlp=tlp,
                 playbook_requested=playbook,
+                tlp=tlp,
                 runtime_configuration=runtime_configuration,
                 tags_labels=tags_labels,
             )
@@ -809,8 +805,8 @@ def _new_analysis_playbook_cli(
             resp = self.send_file_analysis_playbook_request(
                 filename=path.name,
                 binary=path.read_bytes(),
-                tlp=tlp,
                 playbook_requested=playbook,
+                tlp=tlp,
                 runtime_configuration=runtime_configuration,
                 tags_labels=tags_labels,
             )

Diff for: pyintelowl/version.py

@@ -1 +1 @@
-__version__ = "5.0.0-rc"
+__version__ = "5.0.0"