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

Add azure-mgmt-frontdoor to enable WAF functionality #1607

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

jartoo
Copy link
Contributor

@jartoo jartoo commented Jun 19, 2024

SUMMARY

Adding azure-mgmt-frontdoor==1.2.0 (latest version) to enable future pull requests for Azure Frontdoor Web Application Framekwork

Continues to work on #1041 This does not complete this issue, yet.

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

Add azure-mgmt-frontdoor==1.2.0 to requirements.txt to enable Frontdoor WAF in future pull requests.

ADDITIONAL INFORMATION

@Fred-sun
Copy link
Collaborator

@jartoo No module depends on this package, so why add this package?

@Fred-sun Fred-sun added question Further information is requested medium_priority Medium priority work in In trying to solve, or in working with contributors labels Jun 26, 2024
@jartoo
Copy link
Contributor Author

jartoo commented Jul 2, 2024

I was adding this to cover the upcoming azure_rm_frontdoorpolicy.py module, which will cover the WAF functionality for Standard and Premium FD services. I can pull this down and submit with the module itself. I just didn't know if you wanted to vet the separately since it is a new add...

@Fred-sun
Copy link
Collaborator

Fred-sun commented Jul 8, 2024

@jartoo You can develop this dependency and module in a branch so that the purpose of the addition is clearer. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
medium_priority Medium priority question Further information is requested work in In trying to solve, or in working with contributors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants