-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat(arm): AppServiceJavaVersion & FunctionAppsEnableAuthentication #6243
feat(arm): AppServiceJavaVersion & FunctionAppsEnableAuthentication #6243
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great Job!
def __init__(self): | ||
name = "Ensure that 'Java version' is the latest, if used to run the web app" | ||
id = "CKV_AZURE_83" | ||
supported_resources = ['Microsoft.Web/sites'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use a tuple instead of a list.
name = "Ensure that 'Java version' is the latest, if used to run the web app" | ||
id = "CKV_AZURE_83" | ||
supported_resources = ['Microsoft.Web/sites'] | ||
categories = [CheckCategories.GENERAL_SECURITY] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use a tuple instead of a list.
return "site_config/java_version" | ||
|
||
def get_expected_value(self): | ||
return '11' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tsmithv11 Should it still be 11?
(In TF is 11)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like it should be 17 (https://learn.microsoft.com/en-us/azure/app-service/language-support-policy?tabs=linux#jdk-versions-and-maintenance). That TF resource in the TF policy is deprecated which is why it caps out at 11. The new one is 17 (https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/linux_web_app#java_version)
@@ -0,0 +1,82 @@ | |||
# |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove the commented code.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Description
Adds a test
Description
Added a test that checks if a certain field exists under a certain value and checks if the field is equal to a certain value
Fix
How does someone fix the issue in code and/or in runtime?
Checklist: