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

fix: fix address network override via env var #12590

Merged
merged 1 commit into from
Oct 11, 2024

Conversation

masih
Copy link
Member

@masih masih commented Oct 11, 2024

Related Issues

Proposed Changes

Default address.CurrentNetwork was changed to mainnet. As a result if LOTUS_USE_TEST_ADDRESSES is set we need to explicitly use testnet address.

Additional Info

Checklist

Before you mark the PR ready for review, please make sure that:

Default address.CurrentNetwork was changed to mainnet. As a result if
`LOTUS_USE_TEST_ADDRESSES` is set we need to explicitly use testnet
address.
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@masih masih changed the title Fix address network override via env var fix: fix address network override via env var Oct 11, 2024
@masih masih added the skip/changelog This change does not require CHANGELOG.md update label Oct 11, 2024
@github-actions github-actions bot dismissed their stale review October 11, 2024 14:56

PR title now matches the required format.

@masih
Copy link
Member Author

masih commented Oct 11, 2024

I don't know if LOTUS_USE_TEST_ADDRESSES is still used. In any case, this should make the code behave as before.

@masih masih merged commit 63f86a7 into master Oct 11, 2024
95 of 99 checks passed
@masih masih deleted the masih/fix-netaddr-env-override branch October 11, 2024 15:10
rjan90 pushed a commit that referenced this pull request Oct 14, 2024
Default address.CurrentNetwork was changed to mainnet. As a result if
`LOTUS_USE_TEST_ADDRESSES` is set we need to explicitly use testnet
address.
rjan90 pushed a commit that referenced this pull request Oct 14, 2024
Default address.CurrentNetwork was changed to mainnet. As a result if
`LOTUS_USE_TEST_ADDRESSES` is set we need to explicitly use testnet
address.
rjan90 pushed a commit that referenced this pull request Oct 14, 2024
Default address.CurrentNetwork was changed to mainnet. As a result if
`LOTUS_USE_TEST_ADDRESSES` is set we need to explicitly use testnet
address.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip/changelog This change does not require CHANGELOG.md update
Projects
Status: ☑️ Done (Archive)
Development

Successfully merging this pull request may close these issues.

3 participants