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

Use indexer API via API Gateway throughout codebase #12231

Closed
wants to merge 2 commits into from

Conversation

banool
Copy link
Contributor

@banool banool commented Feb 26, 2024

Description

This is part of the migration to API Gateway. More context here: https://www.notion.so/aptoslabs/Migration-1-Legacy-URL-to-API-Gateway-migration-b007742490684b9bab3debd674504a94.

I found the initial files to change by just looking at every file that contained aptoslabs.com.

A previous PR changes the node API URLs.

We can't land this until keyless auth is landed and thoroughly baked in API Gateway.

Test Plan

CI for now.

Copy link

trunk-io bot commented Feb 26, 2024

⏱️ 7m total CI duration on this PR
Job Cumulative Duration Recent Runs
run-tests-main-branch 7m 🟥
file_change_determinator 13s 🟩
permission-check 9s 🟩
permission-check 3s 🟩
permission-check 3s 🟩
permission-check 3s 🟩

🚨 1 job on the last run was significantly faster/slower than expected

Job Duration vs 7d avg Delta
run-tests-main-branch 7m 4m +70%

settingsfeedbackdocs ⋅ learn more about trunk.io

@banool banool force-pushed the banool/new-node-url branch 3 times, most recently from 057ddee to c9efcfa Compare March 1, 2024 19:19
Copy link
Contributor

This issue is stale because it has been open 45 days with no activity. Remove the stale label, comment or push a commit - otherwise this will be closed in 15 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant