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.

Sign up for GitHub

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

Jump to bottom

fix: nil dereference on aggregator retries #1441

Open
wants to merge 3 commits into
base: staging
Choose a base branch
from
Open

fix: nil dereference on aggregator retries #1441

wants to merge 3 commits into from

Commits on Nov 19, 2024

  1. fix: nil dereference on aggregator retries 

    `SendAggregatedResponse` would overwrite `tx`, which in the error case
sets it to `nil`, causing the retry to dereference `nil`.

This fix consists on using separate variables for the simulated tx and
the real one.
    @Oppen
    Oppen committed Nov 19, 2024
    Configuration menu
    Copy the full SHA
    545c003 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'staging' into fix/tx_overwrite

    @Oppen
    Oppen authored Nov 19, 2024
    Configuration menu
    Copy the full SHA
    6198072 View commit details
    Browse the repository at this point in the history

  3. fmt: don't change indent

    @Oppen
    Oppen authored Nov 19, 2024
    Configuration menu
    Copy the full SHA
    cd10c02 View commit details
    Browse the repository at this point in the history