fix: remove debug logs and clean up verbose logging (#25896)
- Remove debug console.log statements in calendar and video adapter services - Clean up verbose request/response logging in OAuth controllers - Remove leftover debug prefixes Ensure only necessary data is captured in observability systems Co-authored-by: Keith Williams <keithwillcode@gmail.com>
This commit is contained in:
1 parent
486817a688
commit
b08997b553
7 files changed
+6
-25
No files matched your search
@@ -75,7 +75,6 @@ export class CalendarsService {
|
||||
eventTypeId: null,
|
||||
prisma: this.dbWrite.prisma as unknown as PrismaClient,
|
||||
});
|
||||
console.log("saving cache", JSON.stringify(result));
|
||||
await this.calendarsCacheService.setConnectedAndDestinationCalendarsCache(userId, result);
|
||||
|
||||
return result;
|
||||
|
||||
+2
-4
@@ -84,9 +84,7 @@ export class OAuthClientUsersController {
|
||||
@Param("clientId") oAuthClientId: string,
|
||||
@Body() body: CreateManagedUserInput
|
||||
): Promise<CreateManagedUserOutput> {
|
||||
this.logger.log(
|
||||
`Creating user with data: ${JSON.stringify(body, null, 2)} for OAuth Client with ID ${oAuthClientId}`
|
||||
);
|
||||
this.logger.log(`Creating user for OAuth Client ${oAuthClientId}`);
|
||||
const client = await this.oauthRepository.getOAuthClient(oAuthClientId);
|
||||
if (!client) {
|
||||
throw new NotFoundException(`OAuth Client with ID ${oAuthClientId} not found`);
|
||||
@@ -133,7 +131,7 @@ export class OAuthClientUsersController {
|
||||
@GetOrgId() organizationId: number
|
||||
): Promise<GetManagedUserOutput> {
|
||||
await this.validateManagedUserOwnership(clientId, userId);
|
||||
this.logger.log(`Updating user with ID ${userId}: ${JSON.stringify(body, null, 2)}`);
|
||||
this.logger.log(`Updating user ${userId} for OAuth Client ${clientId}`);
|
||||
|
||||
const user = await this.oAuthClientUsersService.updateOAuthClientUser(
|
||||
clientId,
|
||||
|
||||
+2
-4
@@ -69,9 +69,7 @@ export class OAuthClientsController {
|
||||
@GetOrgId() organizationId: number,
|
||||
@Body() body: CreateOAuthClientInput
|
||||
): Promise<CreateOAuthClientResponseDto> {
|
||||
this.logger.log(
|
||||
`For organisation ${organizationId} creating OAuth Client with data: ${JSON.stringify(body)}`
|
||||
);
|
||||
this.logger.log(`Creating OAuth Client for organisation ${organizationId}`);
|
||||
|
||||
const organization = await this.teamsRepository.findByIdIncludeBilling(organizationId);
|
||||
if (!organization?.platformBilling || !organization?.platformBilling?.subscriptionId) {
|
||||
@@ -140,7 +138,7 @@ export class OAuthClientsController {
|
||||
@Param("clientId") clientId: string,
|
||||
@Body() body: UpdateOAuthClientInput
|
||||
): Promise<GetOAuthClientResponseDto> {
|
||||
this.logger.log(`For client ${clientId} updating OAuth Client with data: ${JSON.stringify(body)}`);
|
||||
this.logger.log(`Updating OAuth Client ${clientId}`);
|
||||
const client = await this.oAuthClientsService.updateOAuthClient(clientId, body);
|
||||
return { status: SUCCESS_STATUS, data: client };
|
||||
}
|
||||
|
||||
@@ -135,7 +135,6 @@ export class TeamsEventTypesService {
|
||||
});
|
||||
|
||||
const eventType = await this.teamsEventTypesRepository.getEventTypeById(eventTypeId);
|
||||
this.logger.debug("nl debug - update team event type - eventType", JSON.stringify(eventType, null, 2));
|
||||
|
||||
if (!eventType) {
|
||||
throw new NotFoundException(`Event type with id ${eventTypeId} not found`);
|
||||
|
||||
Reference in new issue
Block a user