8000 LogsignSiem Pack PR by content-bot · Pull Request #14565 · demisto/content · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

LogsignSiem Pack PR #14565

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Aug 29, 2021
Merged

LogsignSiem Pack PR #14565

merged 2 commits into from
Aug 29, 2021

Conversation

content-bot
Copy link
Collaborator

Original External PR

external pull request

Contributor

@keremvatandas

Contributing to Cortex XSOAR Content

Make sure to register your contribution by filling the contribution registration form

The Pull Request will be reviewed only after the contribution registration form is filled.

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Description

A few sentences describing the overall goals of the pull request's commits.

Screenshots

Paste here any images that will help the reviewer

Minimum version of Cortex XSOAR

  • 5.5.0
  • 6.0.0
  • 6.1.0
  • 6.2.0

Does it break backward compatibility?

  • Yes
    • Further details:
  • No

Must have

  • Tests
  • Documentation

* created logsignsiem pack

* added logsignsiem classifiers mapper files and deleted dockerfile

* fixed [PA126] validation warning

* fixed some description in yaml file

* deleted override http_request method and updated unittests

* fixed last_fetch parameter and updated tests

* removed unused variable

* fixed logsignsiem api description

* fixed logsignsiem api description

* added query parameter and help section and fixed get-columns-query on api

* fixed unittest func name

* fixed Flake8 error

* Update Packs/LogsignSiem/README.md

* deleted logsign-get-incident method, added default param to query

* rm integration setup from detailed desc

* set default classifier and mapper

Co-authored-by: Itay Keren <ikeren@paloaltonetworks.com>
@content-bot content-bot added docs-approved Contribution Thank you! Contributions are always welcome! labels Aug 29, 2021
@content-bot content-bot requested a review from Itay4 August 29, 2021 06:53
@content-bot
Copy link
Collaborator Author

The CircleCI check from your latest pushed commit was unsuccessful. @content-bot take a look at the build by clicking this link.


Failed Build Steps

  • Validate Files and Yaml

Try and address the listed CircleCI build step failures at your earliest convenience. This will greatly expedite the process of getting your proposed changes merged into master. Happy coding and may the force be with you.

@Itay4 Itay4 merged commit 6d37c12 into master Aug 29, 2021
@Itay4 Itay4 deleted the contrib/keremvatandas_LogsignSiem branch August 29, 2021 11:56
695E
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Contribution Thank you! Contributions are always welcome! docs-approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0