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

Mark bloXroute Max Profit relay as censoring. #600

Merged
merged 2 commits into from
Jul 19, 2024

Conversation

Darkmessage
Copy link
Contributor

Since December 18th 2023, bloXroute Max Profit relay is also censoring OFAC transactions.

This was announced on Twitter here and is now documented on their website:

Both two relays currently propagate all available transactions/bundles except the ones that interacted with addresses sanctioned by OFAC, and they have the same filtering requirement, although for legacy reasons, bloXroute still maintains the two relay names in parallel.

@Darkmessage
Copy link
Contributor Author

I just adjusted the setting that I found during a quick search.

But since I don't know the SmartNode code, maybe some kind of upgrade step is needed, so that the people who use non-regulated profile mode are up-to-date?
But I guess the current relays are always rebuilt and given to the client when restarting the client containers anyway, right?

@Darkmessage
Copy link
Contributor Author

Should also be updated in the docs on the website:

image

So I also created a PR for that.

@Darkmessage Darkmessage marked this pull request as ready for review July 17, 2024 17:22
Copy link
Member

@0xfornax 0xfornax left a comment

Choose a reason for hiding this comment

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

Hey Darkmessage, thanks for the contribution!
I'd just keep current the names instead of using 1 and 2, since their docs still refer to the original names. We can keep the warning that both relays propagate the same txs.

@Darkmessage Darkmessage requested a review from 0xfornax July 19, 2024 04:18
@0xfornax 0xfornax merged commit ed9ae55 into rocket-pool:master Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants