Skip to content

Conversation

@yagreut
Copy link
Contributor

@yagreut yagreut commented Nov 12, 2025

Update AppService schema to support new checks

Checklist

  • I've read the guidelines for contributing to this repository.
  • I've followed the conventions in the PR title.
  • I've added tests that prove my fix is effective or that my feature works.
  • I've updated the documentation with the relevant information (if needed).
  • I've added usage information (if the PR introduces new options)
  • I've included a "before" and "after" example to the description (if the PR is a user interface change).

@github-actions github-actions bot added the apidiff Indicates Go API changes relevant to library consumers (CLI compatibility may be unaffected) label Nov 12, 2025
@github-actions
Copy link

github-actions bot commented Nov 12, 2025

📊 API Changes Detected

Semver impact: major

github.com/aquasecurity/trivy/pkg/iac/scanners/azure
  Compatible changes:
  - Value.GetMetadata: added

github.com/aquasecurity/trivy/pkg/iac/providers/azure/appservice
  Incompatible changes:
  - Service.Authentication: changed from struct{Enabled github.com/aquasecurity/trivy/pkg/iac/types.BoolValue} to Authentication
  - Service.Identity: changed from struct{Type github.com/aquasecurity/trivy/pkg/iac/types.StringValue} to Identity
  - Service.Site: changed from struct{EnableHTTP2 github.com/aquasecurity/trivy/pkg/iac/types.BoolValue; MinimumTLSVersion github.com/aquasecurity/trivy/pkg/iac/types.StringValue} to Site
  Compatible changes:
  - Authentication: added
  - Identity: added
  - Service.HTTPSOnly: added
  - Site: added

@aqua-bot aqua-bot requested a review from a team November 12, 2025 11:53
@nikpivkin nikpivkin added this pull request to the merge queue Nov 19, 2025
Merged via the queue into aquasecurity:main with commit c6d95d7 Nov 19, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

apidiff Indicates Go API changes relevant to library consumers (CLI compatibility may be unaffected)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants