amplify-category-api icon indicating copy to clipboard operation
amplify-category-api copied to clipboard

fix: primary key response should contain all parts of custom claims

Open OperationalFallacy opened this issue 2 months ago • 0 comments

Description of changes

When auth template process graphql responses, it should return all fields in custom claims

CDK / CloudFormation Parameters Changed

Issue https://github.com/aws-amplify/amplify-backend/issues/2973

Description of how you validated changes

run build: yarn build

copied output to the project cp -r ../../amplify-category-api/packages/amplify-graphql-auth-transformer/lib/ node_modules/@aws-amplify/graphql-auth-transformer/lib/

Deployed changes and observed grapqhl contains all parts of custom claims on get/list

{"message":["GraphQL raw response",{"data":{"tradeRequestPK":"roman@xxx::xxx-xxx-xxx-xxx-xxx", ... , "updatedAt":"2025-10-25T18:34:24.511Z"}}]

Checklist

  • [x] PR description included
  • [x] yarn test passes
  • [ ] E2E test run linked
  • [ ] Tests are changed or added
  • [ ] Relevant documentation is changed or added (and PR referenced)
  • [ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
  • [ ] Any CDK or CloudFormation parameter changes are called out explicitly

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This probably requires a more though testing, or a more specialize test to isolate the regression and validate the fix. I also see ownerEntitiesLast stored in snapshot tests etc which I couldn't check or test.

OperationalFallacy avatar Oct 25 '25 19:10 OperationalFallacy