Skip to content

Improve permissions service: caching, nil checks, glob matching #7118

Description

@vhvb1989

Context

Follow-up from PR #7053 review. Parent tracking issue: #7112

PR Conversations

What needs to change

  1. Caching: Add caching to the permissions service — permissions are unlikely to change during an azd session.
  2. Nil checks: Remove unnecessary nil checks before range statements — range over a nil slice is a no-op in Go.
  3. Glob matching: Consider using a glob matching library instead of the custom wildcardMatch implementation to avoid maintaining custom logic if assumptions change.

Files to update

  • cli/azd/pkg/azapi/permissions.go

Metadata

Metadata

Assignees

Labels

area/authAuthentication, credentials, identityarea/securitySecurity, permissions, RBACenhancementNew feature or improvement

Fields

No fields configured for Feature.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions