Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: provided more context in logs #225

Merged

Conversation

frolvanya
Copy link
Contributor

@frolvanya frolvanya commented Feb 4, 2025

Current logs are not as bad as I thought. Relayer just lacked logs when new log is received from EVM (it was shown only after finalization which is about 20mins, so it could be confusing to see no logs when sending a new tx) and I've added some details like signature and tx_hash to some other logs to make debug process easier

closes #200

* chore: added release relayer

* chore: try to adjust repo url

* chore: change workdir

* chore: rename

* chore: bring back correct yml

* chore: update contracts tag trigger

* chore: moved dist to correct place

* fix: deleted musl target
@frolvanya frolvanya merged commit 77c682e into feat/added-message-for-solana-transfers Feb 7, 2025
2 checks passed
frolvanya added a commit that referenced this pull request Feb 7, 2025
* feat: added message field for solana transfers

* feat: provided more context in logs (#225)

* feat: provided more context in logs

* chore: updated omni-types

* chore: retry if attached deposit is too low

* chore: remove event if signature verification failed

* chore: added release relayer (#226)

* chore: added release relayer

* chore: try to adjust repo url

* chore: change workdir

* chore: rename

* chore: bring back correct yml

* chore: update contracts tag trigger

* chore: moved dist to correct place

* fix: deleted musl target

* fix: restart ws connection to rpc when it was dropped and sync lost blocks (#227)
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