Skip to content

Conversation

@ewanharris
Copy link
Contributor

🔧 Changes

Implements a check as to whether the call to retrieve user attribute profiles returns a 403 and return null if so to ensure that tenants without the feature enabled still function without ignoring this resource type.

I have not added the check onto the call in selfServiceProfiles because that explicitly requires adding the property which feels fair to fail on to me.

📚 References

🔬 Testing

Added test and tested manually against tenant

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

@ewanharris ewanharris requested a review from a team as a code owner October 15, 2025 13:29
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 55.55556% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.12%. Comparing base (6596dbb) to head (56be070).

Files with missing lines Patch % Lines
src/tools/auth0/handlers/userAttributeProfiles.ts 55.55% 2 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1181      +/-   ##
==========================================
- Coverage   80.17%   80.12%   -0.06%     
==========================================
  Files         134      134              
  Lines        5100     5107       +7     
  Branches     1014     1016       +2     
==========================================
+ Hits         4089     4092       +3     
- Misses        579      581       +2     
- Partials      432      434       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kushalshit27 kushalshit27 merged commit 4ce981f into master Oct 16, 2025
7 checks passed
@kushalshit27 kushalshit27 deleted the fix/handle-uap-not-enabled branch October 16, 2025 05:54
@kushalshit27 kushalshit27 mentioned this pull request Oct 31, 2025
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.

5 participants