Skip to content

Add audience extraction and validation methods for JWT tokens and Ref…#36

Open
angelmp01 wants to merge 4 commits into
developfrom
chore/marketplace-client-authentication
Open

Add audience extraction and validation methods for JWT tokens and Ref…#36
angelmp01 wants to merge 4 commits into
developfrom
chore/marketplace-client-authentication

Conversation

@angelmp01

Copy link
Copy Markdown
Contributor

…actor authentication token handling in MarketplaceServiceImpl to streamline authentication process.

…actor authentication token handling in MarketplaceServiceImpl to streamline authentication process.
@github-actions

Copy link
Copy Markdown

📊 Static Analysis Summary

🔍 Code Quality Checks

  • ✅ JaCoCo coverage report generated

📦 Download detailed reports from the workflow artifacts.

@angelmp01 angelmp01 requested a review from jorge-romero June 16, 2026 10:33
@angelmp01 angelmp01 marked this pull request as ready for review June 16, 2026 10:36
@github-actions

Copy link
Copy Markdown

📊 Static Analysis Summary

🔍 Code Quality Checks

  • ✅ JaCoCo coverage report generated

📦 Download detailed reports from the workflow artifacts.

}

public static List<String> extractAudiences(Jwt jwt) {
Object audClaim = jwt.getClaims().get("aud");

@jorge-romero jorge-romero Jun 17, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spring provides jwt.getAudience();

you could do something like

List<String> audiences = jwt.getAudience();
if (audiences == null || audiences.isEmpty()) {
    throw new InvalidBearerTokenException("Audience not found in token claims");
}

.....
.....

@github-actions

Copy link
Copy Markdown

📊 Static Analysis Summary

🔍 Code Quality Checks

  • ✅ JaCoCo coverage report generated

📦 Download detailed reports from the workflow artifacts.

…udiences; add tests for client credential flow validation
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

📊 Static Analysis Summary

🔍 Code Quality Checks

  • ✅ JaCoCo coverage report generated

📦 Download detailed reports from the workflow artifacts.

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.

2 participants