Skip to content

Conversation

@imran-ishaq
Copy link
Contributor

@imran-ishaq imran-ishaq commented Oct 2, 2025

Description

Fixed CDI injection qualifier mismatch in FIDO2 metrics services that prevented server startup. Updated Fido2MetricsService and Fido2UserMetricsService to use correct ApplicationFactory.PERSISTENCE_ENTRY_MANAGER_NAME qualifier instead of non-existent custom qualifiers, resolving Weld deployment exceptions and enabling proper FIDO2 server initialization.

Target issue

Resolves FIDO2 service startup failure caused by unsatisfied CDI dependencies in metrics implementation. Fixes HTTP 503/404 errors by correcting PersistenceEntryManager injection qualifiers to match available beans from AppInitializer producer methods.

closes #12312


Test and Document the changes

  • Static code analysis has been run locally and issues have been fixed
  • Relevant unit and integration tests have been added/updated
  • Relevant documentation has been updated if any (i.e. user guides, installation and configuration guides, technical design docs etc)

Please check the below before submitting your PR. The PR will not be merged if there are no commits that start with docs: to indicate documentation changes or if the below checklist is not selected.

  • I confirm that there is no impact on the docs due to the code changes in this PR.

Signed-off-by: imran <imranishaq7071@gmail.com>
@mo-auto
Copy link
Member

mo-auto commented Oct 2, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

@mo-auto mo-auto added comp-jans-fido2 Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality labels Oct 2, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 2, 2025

@imran-ishaq imran-ishaq marked this pull request as ready for review October 2, 2025 14:54
@imran-ishaq imran-ishaq requested a review from yurem as a code owner October 2, 2025 14:54
@yuremm yuremm merged commit 343c1e6 into main Oct 2, 2025
4 checks passed
@yuremm yuremm deleted the jans-fido2-fix-entity-manager-issue branch October 2, 2025 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp-jans-fido2 Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(jans-fido2) : FIDO2 Service Fails to Start Due to CDI Injection Qualifier Mismatch in Metrics Services

5 participants