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

updating threat intelligence allowlist dynamic block #953

Merged

Conversation

cndaan
Copy link
Contributor

@cndaan cndaan commented Jun 3, 2024

Overview/Summary

Updating threat intelligence allowlist dynamic block for the hub spoke configuration. Copied the code from the vwan configuration, that seems to work.

This PR fixes/adds/changes/removes

  1. Now able to update allowlist for threat intelligence in combnation with hub spoke model.
  2. Solves issue Bug report: threat_intelligence_allowlist  #949

Testing evidence

When adding the fqdns or ip_addresses inside the threat_intelligence_allowlist for the firewall resource,I get the following errors:
image

After the fix the values are added:
image

As part of this Pull Request I have

  • Checked for duplicate Pull Requests
  • Associated it with relevant issues, for tracking and closure.
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Performed testing and provided evidence.
  • Updated relevant and associated documentation.

Copy link

@jordyvpaassen jordyvpaassen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approve this code

@matt-FFFFFF
Copy link
Member

Hi thanks for this PR I will get the tests running

@matt-FFFFFF
Copy link
Member

/azp run unit

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@matt-FFFFFF
Copy link
Member

We are preparing for the next major release, I will pull this into main and fix forward.

@matt-FFFFFF matt-FFFFFF merged commit 36df277 into Azure:main Jun 6, 2024
4 of 9 checks passed
anmolnagpal pushed a commit to clouddrove/terraform-azure-landingzone that referenced this pull request Jul 8, 2024
* feat(connectivity): Add option to set allow_non_virtual_wan_traffic in express route gateway. (Azure#914)

Co-authored-by: Miltos Tsatsakis <m.tsatsakis@kaizengaming.com>

* updates to resolve issue Azure#794 (Azure#919)

Co-authored-by: github-actions <action@github.com>

* docs: update docs for threat_intelligence_allowlist (Azure#928)

* Update wiki-sync.yml

* chore(deps): bump github/super-linter from 5 to 6 (Azure#931)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* add link to Upgrade guide from v4.2.0 to v5.0.0 (Azure#934)

Co-authored-by: Matt White <16320656+matt-FFFFFF@users.noreply.github.com>

* Added hub_routing_preference to connectivity advanced configuration (Azure#930)

* Policy sync updates (Azure#959)

* Update Library Templates (automated) (Azure#966)

Co-authored-by: github-actions <action@github.com>

* Add remote branch option (Azure#970)

* Update Library Templates (automated) (Azure#973)

Co-authored-by: github-actions <action@github.com>

* Update Library Templates (automated) (Azure#976)

Co-authored-by: github-actions <action@github.com>

* Remove redundant assignment file (Azure#977)

* updating threat intelligence allowlist dynamic block (Azure#953)

Co-authored-by: Matt White <16320656+matt-FFFFFF@users.noreply.github.com>

* chore(deps): bump azure/powershell from 1 to 2 (Azure#917)

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add OpenSFF Scorecard (Azure#987)

* chore(deps): bump github/codeql-action from 3.24.9 to 3.25.8 (Azure#990)

* chore(deps): bump github.com/hashicorp/go-getter from 1.7.3 to 1.7.4 in /tests/terratest (Azure#986)

* feat!: ama (Azure#968)

* Fix example uami issue (Azure#1000)

* Update Library Templates (automated) (Azure#1001)

Co-authored-by: github-actions <action@github.com>

* docs: additional v6 upgrade detail (Azure#1002)

* Update Library Templates (automated) (Azure#1006)

Co-authored-by: github-actions <action@github.com>

* docs: update docs with FAQ on roadmap and banner for upcoming breaking changes (Azure#1008)

* naming fixed in module

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Mtsa <miltsatsakis@gmail.com>
Co-authored-by: Miltos Tsatsakis <m.tsatsakis@kaizengaming.com>
Co-authored-by: Adam Tuckwell <106317528+ATuckwell@users.noreply.github.com>
Co-authored-by: github-actions <action@github.com>
Co-authored-by: Jared Holgate <jaredholgate@microsoft.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Tobias <tobias-m99@gmx.de>
Co-authored-by: Matt White <16320656+matt-FFFFFF@users.noreply.github.com>
Co-authored-by: QBY-MarkusMaring <106068259+QBY-MarkusMaring@users.noreply.github.com>
Co-authored-by: cae-pr-creator[bot] <126156663+cae-pr-creator[bot]@users.noreply.github.com>
Co-authored-by: Daan Toes <112694691+cndaan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants