Skip to content

fix functionApp matching to allow for other possible Kind values#114

Open
bc0la wants to merge 1 commit into
SpecterOps:mainfrom
bc0la:main
Open

fix functionApp matching to allow for other possible Kind values#114
bc0la wants to merge 1 commit into
SpecterOps:mainfrom
bc0la:main

Conversation

@bc0la

@bc0la bc0la commented Feb 12, 2025

Copy link
Copy Markdown

Fixes an issue (#72) where list-function-apps fails to properly detect Function Apps when the Azure Management API returns a "kind" value like
functionapp,linux

The current code checks for an exact match (functionApp.Kind == "functionapp"), which doesn't account for cases where additional qualifiers (like ,linux) are included. Examples here: https://github.com/Azure/app-service-linux-docs/blob/master/Things_You_Should_Know/kind_property.md

@github-actions

github-actions Bot commented Feb 12, 2025

Copy link
Copy Markdown

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@bc0la

bc0la commented Feb 12, 2025

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

@StranDutton StranDutton self-requested a review January 27, 2026 16:27
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.

1 participant