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

[OTE-626]: Add support for whitelisted addresses (backport #1966) #1976

Merged

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Jul 26, 2024

Changelist

[Describe or list the changes made in this PR]

Test Plan

[Describe how this PR was tested (if applicable)]

Author/Reviewer Checklist

  • If this PR has changes that result in a different app state given the same prior state and transaction list, manually add the state-breaking label.
  • If the PR has breaking postgres changes to the indexer add the indexer-postgres-breaking label.
  • If this PR isn't state-breaking but has changes that modify behavior in PrepareProposal or ProcessProposal, manually add the label proposal-breaking.
  • If this PR is one of many that implement a specific feature, manually label them all feature:[feature-name].
  • If you wish to for mergify-bot to automatically create a PR to backport your change to a release branch, manually add the label backport/[branch-name].
  • Manually add any of the following labels: refactor, chore, bug.

Summary by CodeRabbit

  • New Features

    • Introduced a new configuration parameter for whitelisted addresses to enhance compliance checks.
    • Added a function to validate if an address is whitelisted, impacting geoblocking logic.
    • Enhanced compliance logic to allow whitelisted addresses to bypass certain restrictions.
  • Bug Fixes

    • Improved test coverage for compliance status handling related to address whitelisting and geographical restrictions.
  • Tests

    • Added new test cases to verify the functionality of the whitelisting feature across various scenarios.

This is an automatic backport of pull request #1966 done by [Mergify](https://mergify.com).

Copy link

linear bot commented Jul 26, 2024

Copy link
Contributor

coderabbitai bot commented Jul 26, 2024

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

Share

@Christopher-Li Christopher-Li merged commit 444f4fb into release/indexer/v5.x Jul 26, 2024
11 checks passed
@Christopher-Li Christopher-Li deleted the mergify/bp/release/indexer/v5.x/pr-1966 branch July 26, 2024 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

1 participant