Closed dwasse closed 1 week ago
The recent updates involve integrating a logger for the relayer-db
package and modifying the UpdateQuoteRequestStatus
function to verify valid state transitions within various services related to RFQs (Request for Quotes). Moreover, the core/mapmutex
package has been enhanced with a Keys()
method for easier key retrieval, including respective tests, across several map-related mutex interfaces.
File(s) | Summary |
---|---|
services/rfq/relayer/reldb/base/quote.go |
Added logger and valid state transition checks in UpdateQuoteRequestStatus . Introduced isValidStateTransition . |
services/rfq/relayer/reldb/db.go |
Modified UpdateQuoteRequestStatus method signature to include prevStatus . |
services/rfq/relayer/service/chainindexer.go |
Updated UpdateQuoteRequestStatus calls to include additional prevStatus argument. |
services/rfq/relayer/service/handlers.go |
Enhanced UpdateQuoteRequestStatus usages in various handler functions to pass additional status arguments. |
services/rfq/relayer/service/relayer.go |
Adjusted processRequest for improved status updating logic with status reference passing. |
services/rfq/relayer/service/statushandler.go |
Updated deadlineMiddleware to include req.Status in UpdateQuoteRequestStatus calls. |
core/mapmutex/mapmutex.go |
Added Keys() method to untypedMapMutex and implemented in untypedMapMutexImpl for debugging purposes. |
core/mapmutex/mapmutex_test.go |
Introduced tests for key retrieval functionalities in various map mutex types. |
core/mapmutex/stringer.go |
Added Keys() method to StringerMapMutex , StringMapMutex , and IntMapMutex interfaces, and provided implementations. |
A logger’s tale, a journey to refine,
Quote requests now traverse a secure line.
State transitions verified, errors foregone,
Keys unlocked, mutexes carry on.
Debugging’s a breeze, status held tight,
In RFQ’s realm, transitions bright. 🌟🐇
[!TIP]
Early access features: enabled
We are currently testing the following features in early access: - **OpenAI `gpt-4o` model for code reviews and chat**: OpenAI claims that this model is better at understanding and generating code than the previous models. We seek your feedback over the next few weeks before making it generally available. Note: - You can enable or disable early access features from the CodeRabbit UI or by updating the CodeRabbit configuration file. - Please join our [Discord Community](https://discord.com/invite/GsXnASn26c) to provide feedback and report issues. - OSS projects are currently opted into early access features by default.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 40.00000%
with 45 lines
in your changes missing coverage. Please review.
Project coverage is 25.64578%. Comparing base (
0d573f2
) to head (b140cb2
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Latest commit: |
b140cb2
|
Status: | ✅ Deploy successful! |
Preview URL: | https://1e17f5b9.sanguine-fe.pages.dev |
Branch Preview URL: | https://feat-restrict-state-tx.sanguine-fe.pages.dev |
Summary by CodeRabbit
New Features
Bug Fixes
UpdateQuoteRequestStatus
function to check for valid state transitions and log any invalid transitions.Tests