Skip to content

Conversation

@arcoraven
Copy link
Contributor

@arcoraven arcoraven commented Dec 11, 2024

PR-Codex overview

This PR introduces functionality for managing backend wallet access in a Lite version, including creation, retrieval, and updating of wallet access records. It also enhances server routing and logging mechanisms.

Detailed summary

  • Added getBackendWalletLiteAccess function to retrieve wallet access.
  • Implemented createBackendWalletLiteAccess function for wallet access creation.
  • Developed updateBackendWalletLiteAccess function for updating wallet access.
  • Registered new routes for creating and listing backend wallet access.
  • Updated schema.prisma to include BackendWalletLiteAccess model.
  • Created migration for new database table backend_wallet_lite_access.
  • Enhanced generateLocalWallet to accept an encryption password.
  • Improved logging middleware to filter sensitive log paths.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

"/backend-wallet/lite",
]);

function shouldLog(request: FastifyRequest) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Small changes here to add the POST /backend-wallet/lite path to not log the litePassword

const { method, routeOptions, headers, params, query, body } = request;
const { statusCode, elapsedTime } = reply;
const isError = statusCode >= 400;
if (shouldLog(request)) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Random whitespace changes in this file. I confirmed its using spaces now, so I must have saved it with tabs in the past. (Grrr Biome)

@arcoraven arcoraven changed the base branch from main to lite-beta December 11, 2024 09:51
@arcoraven arcoraven merged commit e4e5dda into lite-beta Dec 11, 2024
5 of 6 checks passed
@arcoraven arcoraven deleted the ph/lite1-crud branch December 11, 2024 09:51
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