Skip to content
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

Phishlabs fetch incidents bug #29206

Merged
merged 16 commits into from Sep 28, 2023
Merged

Phishlabs fetch incidents bug #29206

merged 16 commits into from Sep 28, 2023

Conversation

eyalpalo
Copy link
Contributor

@eyalpalo eyalpalo commented Aug 24, 2023

Status

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

Related Issues

fixes: link to the issue

Description

There was a bug with fetch as last run was not assigned with a value when no incidents were fetched.

Must have

  • Tests
  • Documentation

@eyalpalo eyalpalo changed the title added logs and last id duplication prevention for Phislabs integration Phishlabs fetch incidents bug Sep 26, 2023
@eyalpalo eyalpalo marked this pull request as ready for review September 26, 2023 12:59
eyalpalo and others added 2 commits September 27, 2023 16:08
Co-authored-by: Dan Tavori <38749041+dantavori@users.noreply.github.com>
@eyalpalo
Copy link
Contributor Author

Need to be forced merged, fails because we dont have creds for this integration. @dantavori

@eyalpalo eyalpalo merged commit bff3ce3 into master Sep 28, 2023
16 of 17 checks passed
@eyalpalo eyalpalo deleted the xsup_27359 branch September 28, 2023 11:23
wolyslager pushed a commit to wolyslager/content that referenced this pull request Sep 29, 2023
* added logs and last id duplication prevention

* fix

* revert

* fix

* bug_fix

* improv

* removed logs

* rn

* unit test

* pre commit

* Update Packs/PhishLabs/ReleaseNotes/1_1_16.md

Co-authored-by: Dan Tavori <38749041+dantavori@users.noreply.github.com>

* CR

* ignore

---------

Co-authored-by: Dan Tavori <38749041+dantavori@users.noreply.github.com>
wolyslager pushed a commit to wolyslager/content that referenced this pull request Oct 2, 2023
* added logs and last id duplication prevention

* fix

* revert

* fix

* bug_fix

* improv

* removed logs

* rn

* unit test

* pre commit

* Update Packs/PhishLabs/ReleaseNotes/1_1_16.md

Co-authored-by: Dan Tavori <38749041+dantavori@users.noreply.github.com>

* CR

* ignore

---------

Co-authored-by: Dan Tavori <38749041+dantavori@users.noreply.github.com>
xsoar-bot pushed a commit to xsoar-contrib/content that referenced this pull request Oct 5, 2023
* added logs and last id duplication prevention

* fix

* revert

* fix

* bug_fix

* improv

* removed logs

* rn

* unit test

* pre commit

* Update Packs/PhishLabs/ReleaseNotes/1_1_16.md

Co-authored-by: Dan Tavori <38749041+dantavori@users.noreply.github.com>

* CR

* ignore

---------

Co-authored-by: Dan Tavori <38749041+dantavori@users.noreply.github.com>
sapirshuker pushed a commit that referenced this pull request Dec 21, 2023
* added logs and last id duplication prevention

* fix

* revert

* fix

* bug_fix

* improv

* removed logs

* rn

* unit test

* pre commit

* Update Packs/PhishLabs/ReleaseNotes/1_1_16.md

Co-authored-by: Dan Tavori <38749041+dantavori@users.noreply.github.com>

* CR

* ignore

---------

Co-authored-by: Dan Tavori <38749041+dantavori@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants