Don't always log headers

This commit is contained in:
Wolfgang Kulhanek
2025-10-16 11:44:23 +02:00
parent 9609cbd7f0
commit 7da0bc627c
2 changed files with 9 additions and 3 deletions

View File

@@ -611,7 +611,8 @@ function server(
clock, clock,
i8n, i8n,
serverOpts.smapiAuthTokens, serverOpts.smapiAuthTokens,
serverOpts.smapiTokenStore serverOpts.smapiTokenStore,
serverOpts.logRequests
); );
if (serverOpts.applyContextPath) { if (serverOpts.applyContextPath) {

View File

@@ -405,7 +405,8 @@ function bindSmapiSoapServiceToExpress(
clock: Clock, clock: Clock,
i8n: I8N, i8n: I8N,
smapiAuthTokens: SmapiAuthTokens, smapiAuthTokens: SmapiAuthTokens,
tokenStore: SmapiTokenStore tokenStore: SmapiTokenStore,
logRequests: boolean
) { ) {
const sonosSoap = new SonosSoap(bonobUrl, linkCodes, smapiAuthTokens, clock, tokenStore); const sonosSoap = new SonosSoap(bonobUrl, linkCodes, smapiAuthTokens, clock, tokenStore);
@@ -424,6 +425,7 @@ function bindSmapiSoapServiceToExpress(
// Check if token/key is associated with a user // Check if token/key is associated with a user
const smapiToken = sonosSoap.getCredentialsForToken(credentials.loginToken.token); const smapiToken = sonosSoap.getCredentialsForToken(credentials.loginToken.token);
if (!smapiToken) { if (!smapiToken) {
logger.warn("Token not found in store - possibly old/expired token from Sonos cache. Try removing and re-adding the service in Sonos app.");
return E.left(new InvalidTokenError("Token not found")); return E.left(new InvalidTokenError("Token not found"));
} }
@@ -431,6 +433,7 @@ function bindSmapiSoapServiceToExpress(
const effectiveKey = credentials.loginToken.key || smapiToken.key; const effectiveKey = credentials.loginToken.key || smapiToken.key;
if (smapiToken.key !== effectiveKey) { if (smapiToken.key !== effectiveKey) {
logger.warn("Token key mismatch", { storedKey: smapiToken.key, providedKey: effectiveKey });
return E.left(new InvalidTokenError("Token key mismatch")); return E.left(new InvalidTokenError("Token key mismatch"));
} }
@@ -789,7 +792,9 @@ function bindSmapiSoapServiceToExpress(
logger.debug( logger.debug(
`Fetching metadata type=${type}, typeId=${typeId}, acceptLanguage=${acceptLanguage}` `Fetching metadata type=${type}, typeId=${typeId}, acceptLanguage=${acceptLanguage}`
); );
console.log("getMetadata headers", headers) if (logRequests) {
console.log("getMetadata headers", headers);
}
const lang = i8n(...asLANGs(acceptLanguage)); const lang = i8n(...asLANGs(acceptLanguage));
const albums = (q: AlbumQuery): Promise<GetMetadataResponse> => const albums = (q: AlbumQuery): Promise<GetMetadataResponse> =>