Open
Conversation
|
@Obiajulu-gif is attempting to deploy a commit to the missblue00's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@Obiajulu-gif Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
Owner
|
@Obiajulu-gif the ci/cd checks failed and there are merge conflicts, please resolve |
- Happy path: Registration -> Payment -> Confirmation -> Mint - Error paths: invalid signatures, insufficient balances, rate limits - Uses Jest and Supertest with real JwtAuthGuard/JwtStrategy - Install jsonwebtoken as dev dep for test token generation
Adds StellarService to payments-engine for blockchain transfers. Introduces a scheduled worker in the API to process pending redemptions using a mock Repository pattern until Prisma is fully integrated. Closes MissBlue00#20.
Author
|
@MissBlue00 I have fix the error merge it |
Obiajulu-gif
force-pushed
the
setup_k8s_manifests
branch
from
dc06161 to
3e7a346
Compare
Author
|
@MissBlue00 please merge my code |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
This pull request introduces a comprehensive observability and structured logging system to the
stellar-payAPI service. It adds structured JSON logging with correlation IDs, integrates logging and request context throughout the API, and provides CloudWatch dashboard assets for production monitoring. The changes also include middleware, interceptors, and exception filters to ensure consistent logging and traceability across requests and errors.Observability and Logging Enhancements:
StructuredLogger) with support for log levels, correlation IDs, and child loggers for contextual logging. Added a test suite for logger behavior. (apps/api/src/observability/logger.ts,apps/api/src/observability/logger.spec.ts) [1] [2]AsyncLocalStorage, with middleware to generate and attach correlation IDs to each request. (apps/api/src/observability/request-context.ts,apps/api/src/observability/request-context.middleware.ts) [1] [2]apps/api/src/observability/request-logging.interceptor.ts,apps/api/src/observability/logging-exception.filter.ts) [1] [2]apps/api/src/app.module.ts,apps/api/src/main.ts,apps/api/src/observability/index.ts) [1] [2] [3] [4] [5]Application Logging Integration:
AppController,AppService,HealthController,TreasuryController,TreasuryService) to emit structured events for important actions and API endpoints. (apps/api/src/app.controller.ts,apps/api/src/app.service.ts,apps/api/src/health/health.controller.ts,apps/api/src/treasury/treasury.controller.ts,apps/api/src/treasury/treasury.service.ts) [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]Documentation and Configuration:
stellar-pay/README.md)apps/api/.env.example)These changes significantly improve the API's observability, making it easier to trace requests, debug issues, and monitor production health.
Closes #93