Frontend sensitive data removal #216
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Remove console.log statements that could expose sensitive information: - Cryptographic hashes and signatures in inviteUtils.ts - Signature and hash data in HuntDetails.tsx team invite flow - Stream tokens in HuddleRoom.tsx - Request body with user addresses in Clue.tsx - Full transaction receipts in transactionUtils.ts Also cleaned up verbose debug logs that were cluttering the console: - Network/contract debug info in Create.tsx and Hunts.tsx - Transaction args logs - Full error object serialization Error logging (console.error) for debugging purposes is retained. Closes #202
|
Cursor Agent can help with this pull request. Just |
👷 Deploy Preview for khoj-alpha processing.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Removes sensitive log statements and unnecessary debug logs from the frontend to enhance security and reduce console clutter.
The removed logs included potentially sensitive information such as hash values, signatures, encoded invite data, stream tokens/keys, request bodies with user addresses, full transaction receipts, and verbose contract debug information. Removing these prevents accidental exposure of private data and improves the clarity of the console for legitimate debugging.