lotus-gateway: Unfilter Filecoin.EthGetFilterLogs

[sudo-shashank]

Summary of changes

Changes introduced in this pull request:

• Unfilter Filecoin.EthGetFilterLogs check with gateway as it's resolved in lotus release v1.35.1.

Reference issue to close (if applicable)

Closes #6580

Other information and links

filecoin-project/lotus#13519

Change checklist

• I have performed a self-review of my own code,
• I have made corresponding changes to the documentation. All new code adheres to the team's documentation standards,
• I have added tests that prove my fix is effective or that my feature works (if possible),
• I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

Outside contributions

• I have read and agree to the CONTRIBUTING document.
• I have read and agree to the AI Policy document. I understand that failure to comply with the guidelines will lead to rejection of the pull request.

Summary by CodeRabbit

• Tests
  • Updated test configuration to reflect that an API method is no longer marked as unsupported.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 0f4618cd-8373-4dd5-b6f8-eb4c6a464c2d

📥 Commits

Reviewing files that changed from the base of the PR and between d52a294 and f2256ee.

📒 Files selected for processing (1)

• scripts/tests/api_compare/filter-list-gateway

💤 Files with no reviewable changes (1)

• scripts/tests/api_compare/filter-list-gateway

---

Walkthrough

This pull request removes Filecoin.EthGetFilterLogs from the "broken" methods list in the lotus-gateway API comparison filter configuration, reflecting that this RPC method has been fixed and is now functional with the gateway.

Changes

Cohort / File(s)	Summary
Gateway Filter Configuration scripts/tests/api_compare/filter-list-gateway	Removed !Filecoin.EthGetFilterLogs from the broken methods section, indicating the method is no longer marked as unsupported.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• test: add lotus-gateway parity test #6570: Manages entries in the same filter list for Filecoin API methods and originally added EthGetFilterLogs to the broken list.
• fix eth_get_filter_logs test #6363: Updates how the eth_get_filter_logs test is handled, removing test ignores that relate to the same method.
• fix: RPC parity test failure #6813: Modifies the same gateway filter list, adjusting broken method entries.

Suggested reviewers

• hanabi1224
• akaladarshi
• LesnyRumcajs

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: removing the filter on Filecoin.EthGetFilterLogs for lotus-gateway, which aligns with the code change and PR objectives.
Linked Issues check	✅ Passed	The PR successfully addresses issue #6580 by removing Filecoin.EthGetFilterLogs from the gateway filter list, reflecting the fix in lotus v1.35.1 as intended.
Out of Scope Changes check	✅ Passed	The change is narrowly scoped to removing one filter entry from the gateway test configuration, directly addressing the linked issue with no extraneous modifications.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch shashank/fix-eth_getFilterLogs

✨ Simplify code

• Create PR with simplified code
• Commit simplified code in branch shashank/fix-eth_getFilterLogs

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.89%. Comparing base (d52a294) to head (f2256ee).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

see 14 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d52a294...f2256ee. Read the comment docs.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.