Skip to content

[Bug]: Linea Send transactions remain in a Pending state indefinitely unless they are sped up. #31176

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

Closed
sleepytanya opened this issue Mar 20, 2025 · 3 comments
Assignees
Labels
needs-triage Issue needs to be triaged Sev2-normal Normal severity; minor loss of service or inconvenience. team-confirmations Push issues to confirmations team team-linea Linea team type-bug Something isn't working

Comments

@sleepytanya
Copy link
Contributor

sleepytanya commented Mar 20, 2025

Describe the bug

Linea Send transactions remain in a Pending state indefinitely unless they are sped up.
Some transactions can be confirmed, but other will stuck Pending.
(STX are not available on Linea)
Gas API response:
https://gas.api.cx.metamask.io/networks/59144/suggestedGasFees

No errors in the network tab:

Image

Expected behavior

Transactions are processed in a timely manner without requiring them to be sped up

Screenshots/Recordings

Screen.Recording.2025-03-20.at.17.03.25.mov
Image Image

Steps to reproduce

  1. Connect to Linea mainnet
  2. Start Send transaction

Error messages or log output

Detection stage

In production (default)

Version

12.14.1

Build type

None

Browser

Chrome

Operating system

MacOS

Hardware wallet

No response

Additional context

No response

Severity

No response

@sleepytanya sleepytanya added regression-prod-12.4.1 Regression bug that was found in production in release 12.4.1 Sev2-normal Normal severity; minor loss of service or inconvenience. team-confirmations Push issues to confirmations team team-linea Linea team type-bug Something isn't working labels Mar 20, 2025
@github-project-automation github-project-automation bot moved this to To be fixed in Bugs by severity Mar 20, 2025
@github-project-automation github-project-automation bot moved this to To be triaged in Bugs by team Mar 20, 2025
@metamaskbot metamaskbot added needs-triage Issue needs to be triaged regression-prod-12.14.1 Regression bug that was found in production in release 12.14.1 and removed regression-prod-12.4.1 Regression bug that was found in production in release 12.4.1 labels Mar 20, 2025
@sleepytanya sleepytanya changed the title [Bug]: Linear Send transactions remain in a Pending state indefinitely unless they are sped up. [Bug]: Linea Send transactions remain in a Pending state indefinitely unless they are sped up. Mar 20, 2025
@hesterbruikman
Copy link

Removing the regression label here. From Slack conversations with Linea team, this is an effect of market conditions rather than tied to a release version.

@hesterbruikman hesterbruikman removed the regression-prod-12.14.1 Regression bug that was found in production in release 12.14.1 label Mar 28, 2025
github-merge-queue bot pushed a commit that referenced this issue Apr 1, 2025
…oller` config (#31476)

<!--
Please submit this PR as a draft initially.
Do not mark it as "Ready for review" until the template has been
completely filled out, and PR status checks have passed at least once.
-->

## **Description**

<!--
Write a short description of the changes included in this pull request,
also include relevant motivation and context. Have in mind the following
questions:
1. What is the reason for the change?
2. What is the improvement/solution?
-->

This PR aims to enable update of `txParams` gas updates on every
`GasFeeController` update.

[![Open in GitHub
Codespaces](https://github.com/codespaces/badge.svg)](https://codespaces.new/MetaMask/metamask-extension/pull/31476?quickstart=1)

## **Related issues**

Fixes: MetaMask/MetaMask-planning#4519
Might be related:
#31176

## **Manual testing steps**

N/A

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->

### **After**

<!-- [screenshots/recordings] -->

## **Pre-merge author checklist**

- [X] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask
Extension Coding
Standards](https://github.com/MetaMask/metamask-extension/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [X] I've completed the PR template to the best of my ability
- [X] I’ve included tests if applicable
- [X] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [X] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-extension/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.
@sleepytanya
Copy link
Contributor Author

@OGPoyraz
Gas prices atm are not very high, and I can't get stuck tx. Setting low advanced gas values makes the tx fail with the Gas price below configured minimum gas price(but it doesn't stay Pending forever), this is main:

Image

Interesting that I see the same behavior in the RC, may be it makes sense to wait for the higher Linea gas?

@OGPoyraz
Copy link
Member

OGPoyraz commented Apr 2, 2025

I want to summarise what happens in background.

We have a special gas fee flow on Linea.
https://github.com/MetaMask/core/blob/main/packages/transaction-controller/src/gas-flows/LineaGasFeeFlow.ts

Basically what we do is that asking gas estimations via linea_estimateGas RPC method and depending on the response, we multiply that value to determine low medium high values.
https://github.com/MetaMask/core/blob/main/packages/transaction-controller/src/gas-flows/LineaGasFeeFlow.ts#L36

Even if you select low (will be multiplied by 1) in a congested network, it should mark correct gas values on submitted transaction (after this fix). It should take some time, but eventually should be confirmed now.

I am adding a recording that it succesfully confirmed transaction on medium and low values.

Screen.Recording.2025-04-02.at.09.26.43.mov

But I am not sure why we see the error above, because it's an RPC error and I wonder in what circumstances they throw it. If you find consistent way to get this error, please open another task so we can investigate further with Linea team.

For this issue - I am considering that it's done as recording shows that it works as expected, again feel free to re-open if it consistently stays on pending on your end with low values.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-triage Issue needs to be triaged Sev2-normal Normal severity; minor loss of service or inconvenience. team-confirmations Push issues to confirmations team team-linea Linea team type-bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

4 participants