fix(authz): access pdp should use proto getter (#2530)

jakedoublev · web-flow · commit f85621280954 · 2025-07-07T17:22:10.000Z
### Proposed Changes

* Fixes one direct access of proto fields without using nil-safe getter

### Checklist

- [ ] I have added or updated unit tests
- [ ] I have added or updated integration tests (if appropriate)
- [ ] I have added or updated documentation

### Testing Instructions
diff --git a/service/internal/access/v2/pdp.go b/service/internal/access/v2/pdp.go
@@ -116,7 +116,7 @@ func NewPolicyDecisionPoint(
 		mappedValue := sm.GetAttributeValue()
 		mappedValueFQN := mappedValue.GetFqn()
 		if _, ok := allEntitleableAttributesByValueFQN[mappedValueFQN]; ok {
-			allEntitleableAttributesByValueFQN[mappedValueFQN].Value.SubjectMappings = append(allEntitleableAttributesByValueFQN[mappedValueFQN].Value.SubjectMappings, sm)
+			allEntitleableAttributesByValueFQN[mappedValueFQN].GetValue().SubjectMappings = append(allEntitleableAttributesByValueFQN[mappedValueFQN].GetValue().GetSubjectMappings(), sm)
 			continue
 		}
 		// Take subject mapping's attribute value and its definition from memory

Original file line number	Diff line number	Diff line change
`@@ -116,7 +116,7 @@ func NewPolicyDecisionPoint(`
`116`	`116`	`mappedValue := sm.GetAttributeValue()`
`117`	`117`	`mappedValueFQN := mappedValue.GetFqn()`
`118`	`118`	`if _, ok := allEntitleableAttributesByValueFQN[mappedValueFQN]; ok {`
`119`		`- allEntitleableAttributesByValueFQN[mappedValueFQN].Value.SubjectMappings = append(allEntitleableAttributesByValueFQN[mappedValueFQN].Value.SubjectMappings, sm)`
	`119`	`+ allEntitleableAttributesByValueFQN[mappedValueFQN].GetValue().SubjectMappings = append(allEntitleableAttributesByValueFQN[mappedValueFQN].GetValue().GetSubjectMappings(), sm)`
`120`	`120`	`continue`
`121`	`121`	`}`
`122`	`122`	`// Take subject mapping's attribute value and its definition from memory`