Skip to content

Debug/logs#38

Open
gabrielmouallem wants to merge 3 commits intomasterfrom
debug/logs
Open

Debug/logs#38
gabrielmouallem wants to merge 3 commits intomasterfrom
debug/logs

Conversation

@gabrielmouallem
Copy link
Contributor

@gabrielmouallem gabrielmouallem commented Nov 14, 2025

Note

Add extensive debug logging across HTTP methods (error handling, PATCH requests, URL building) and update exports.

  • lib/http-methods:
    • Error handling: Add console logs in handleError for the raw error and err.response.body before parsing.
    • PATCH requests: Log path, headers, data, and response body in patchRequest.
    • URL building: Log path, searchParams, and the final URL in buildUrl.
    • Exports: Add line exporting deleteRequest via exports.setAuthHeaders = deleteRequest.

Written by Cursor Bugbot for commit 25b7a7d. This will update automatically on new commits. Configure here.


const url = process.env.LATITUDE_API_DOMAIN || 'https://api.latitude.sh';

console.log(url + path + searchParams);
Copy link

Choose a reason for hiding this comment

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

Bug: Temporary debug logs: Codebase pollution.

Multiple console.log debugging statements were added throughout the file (in handleError, patchRequest, and buildUrl functions). These debug logs appear to be temporary debugging code that should not be committed to the codebase as they add noise to production logs and may leak sensitive information like headers and error details.

Fix in Cursor Fix in Web

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