fix(console): app details

[eliobischof]

The app details for OIDC apps show the SAML config and URLs. In the OIDC apps URLs section, the OIDC URLs are not even listed, only the useless SAML URLs.

This PR fixes this bug introduced with #7572.

The following screenshots show the new details pages.

API

OIDC

SAML

Definition of Ready

• I am happy with the code
• Short description of the feature/issue is added in the pr description
• PR is linked to the corresponding user story
• Acceptance criteria are met
• All open todos and follow ups are defined in a new ticket and justified
• Deviations from the acceptance criteria and design are agreed with the PO and documented.
• No debug or dead code
• My code has no repetitions
• Critical parts are tested automatically
• Where possible E2E tests are implemented
• Documentation/examples are up-to-date
• All non-functional requirements are met
• Functionality of the acceptance criteria is checked manually on the dev system.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 29, 2024 10:11am

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 62.79%. Comparing base (cc56eea) to head (341f91f).
Report is 405 commits behind head on main.

❗ Current head 341f91f differs from pull request most recent head 1cb718e. Consider uploading reports for the commit 1cb718e to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #7827       +/-   ##
===========================================
+ Coverage   50.10%   62.79%   +12.68%     
===========================================
  Files        1244     1312       +68     
  Lines      117337   108839     -8498     
===========================================
+ Hits        58788    68342     +9554     
+ Misses      55594    36567    -19027     
- Partials     2955     3930      +975     

Flag	Coverage Δ
core-integration-tests-postgres	62.79% <ø> (+38.36%)	⬆️
core-unit-tests	?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[livio-a]

works, but IMO we should add at least the introspection_endpoint and jwks_uri and maybe even revocation_endpoint for APIs.

[eliobischof]

works, but IMO we should add at least the introspection_endpoint and jwks_uri and maybe even revocation_endpoint for APIs.

I mapped the URLs explicitly, fixed their order, added the mentioned URLs to the API and prettified the keys.

[livio-a]

sorry apparently overlooked that before, but just noted that the issuer is missing on the OIDC configs
and wouldn't it make sense to have the entityID / metadata endpoint on the SAML config, resp. could we even use that for the link instead of well-known? maybe also any input on that from @stebenz?

[github-actions]

🎉 This PR is included in version 2.51.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀