Skip to content

Commit

Permalink
Fixing limit argument for mati-feed-get-indicators (#29995) (#30057)
Browse files Browse the repository at this point in the history
* Fixing `limit` argument for `mati-feed-get-indicators`

* Addressing feedback

* Bump Docker version

* Marking value as required

* Switching to `int` due to typing validations

Co-authored-by: Christopher Hultin <chrishultin@google.com>
  • Loading branch information
content-bot and chrishultin committed Oct 5, 2023
1 parent a2fea44 commit 60fd04e
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 3 deletions.
Expand Up @@ -1112,7 +1112,8 @@ def fetch_indicators(client: MandiantClient, args: dict = None) -> tuple[List, d
if not args:
args = {}

limit = args.get("limit", client.limit)
limit = int(args.get("limit", client.limit))

# Cap maximum number of indicators to 1000
if limit > 1000:
limit = 1000
Expand Down
Expand Up @@ -485,7 +485,7 @@ script:
- Actors
required: true
description: Get Mandiant Indicators.
dockerimage: demisto/python3:3.10.13.74666
dockerimage: demisto/python3:3.10.13.75921
feed: true
runonce: false
script: '-'
Expand Down
@@ -0,0 +1,6 @@

#### Integrations

##### Mandiant Advantage Threat Intelligence

- Fixed an issue where *limit* argument from ***mati-feed-get-indicators*** command was not parse properly.
Expand Up @@ -2,7 +2,7 @@
"name": "Mandiant Advantage Threat Intelligence",
"description": "Integrate your Mandiant Advantage Threat Intelligence data with Cortex XSOAR",
"support": "partner",
"currentVersion": "1.0.8",
"currentVersion": "1.0.9",
"author": "Mandiant",
"hidden": false,
"url": "https://www.mandiant.com/support",
Expand Down

0 comments on commit 60fd04e

Please sign in to comment.