Skip to content

Removing these log messages as customer complaining of verbose logging #212

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
dxasupport merged 1 commit into from
Jan 22, 2025
Merged

Removing these log messages as customer complaining of verbose logging #212

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 0 additions & 5 deletions ...-tridion-common/src/main/java/com/sdl/dxa/tridion/pcaclient/DefaultApiClientProvider.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -82,9 +82,6 @@ public void removeGlobalClaim(ClaimValue claim) {
@Override
public ApiClient getClient() {
ClaimStore claimStore = AmbientDataContext.getCurrentClaimStore();
if (claimStore == null) {
log.debug("No claimstore found (is the ADF module configured in the Web.Config?) so unable to populate claims for PCA.");
}

String previewToken = getClaimValue(WebClaims.REQUEST_HEADERS, X_PREVIEW_SESSION_TOKEN,
claim -> Optional.of(((List<String>) claim).get(0)))
Expand Down Expand Up @@ -113,12 +110,10 @@ public ApiClient getClient() {
}

if (!configurationLoader.claimForwarding()) {
log.debug("The claimstore is not available so no claim forwarding from claimstore will be performed. Make sure the ADF module is configured in the Web.Config to enable this option.");
return client;
}

if (claimStore == null) {
log.debug("The claimstore is not available so no claim forwarding from claimstore will be performed. Make sure the ADF module is configured in the Web.Config to enable this option.");
return client;
}

Expand Down
1 change: 0 additions & 1 deletion ...provider/src/main/java/com/sdl/webapp/tridion/contextengine/AdfContextClaimsProvider.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,6 @@ protected Map<URI, Object> getCurrentClaims() {
ClaimStore currentClaimStore = AmbientDataContext.getCurrentClaimStore();
if (currentClaimStore == null)
{
log.warn("Claimstore is not available. Check the AmbientFrameworkModule is configured in Web.Config");
return Collections.emptyMap();
}
return currentClaimStore.getAll();
Expand Down
Loading