Skip to content

Comments

Qa#577

Merged
NikolayRn merged 23 commits intomasterfrom
qa
Oct 21, 2025
Merged

Qa#577
NikolayRn merged 23 commits intomasterfrom
qa

Conversation

@LiorAgnin
Copy link
Member

subj

LiorAgnin and others added 3 commits September 9, 2025 13:45
…Service

- Enhanced logging for JSON-RPC errors in BundlerApiInterceptor, including specific messages for validation failures and CALL_EXCEPTION errors.
- Updated error handling in PaymasterApiService to log service unavailability errors (502, 503) for better debugging.
fix: Improve error handling in BundlerApiInterceptor and PaymasterApiService
@LiorAgnin LiorAgnin requested a review from NikolayRn September 9, 2025 10:55
JackRooty and others added 20 commits September 9, 2025 15:19
- Updated error handling in SmartWalletsAAService, SmartWalletsLegacyService, and RelayAPIService to use RpcException for better microservice context.
- Improved robustness of error messages by checking for existing properties before accessing them.
- Ensured consistent error handling across services to enhance debugging and maintainability.
fix: Enhance error handling in Smart Wallet services
…-proxy

- Added a logger to RelayAPIService to capture and log errors more effectively.
- Enhanced error handling in client-proxy to include statusCode and message for better clarity in exceptions.
- Ensured consistent error messaging across services for improved debugging.
fix: Improve error logging and handling in RelayAPIService and client-proxy
- Updated error responses in SmartWalletsAAService and SmartWalletsLegacyService to use 'error' and 'status' fields for consistency.
- Enhanced error handling in AllExceptionsFilter and client-proxy to accommodate new error structure.
- Improved logging for better debugging and clarity in error messages across services.
fix: Standardize error handling in Smart Wallet services
Copy link
Contributor

@NikolayRn NikolayRn left a comment

Choose a reason for hiding this comment

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

LGTM

@NikolayRn NikolayRn merged commit 5a45e4a into master Oct 21, 2025
7 of 13 checks passed
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.

3 participants