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

Add status of signing and filtering for everybody #397

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kroeckx
Copy link

@kroeckx kroeckx commented May 21, 2020

The signing is based on what https://rpki.cloudflare.com/?view=bgp
returns. It's probably not ideal, but it's the best I know.
I have used:
< 10% valid signatures: not signing

90% valid signatures: signing
otherwise: partial signing

The signing is based on what https://rpki.cloudflare.com/?view=bgp
returns. It's probably not ideal, but it's the best I know.
I have used:
< 10% valid signatures: not signing
> 90% valid signatures: signing
otherwise: partial signing
@kroeckx
Copy link
Author

kroeckx commented May 30, 2020

Do you intend to merge this? It now of course contains a merge conflict since you've merged something. I don't intend to fix the merge conflict unless you want to merge this.

@jejenone
Copy link
Contributor

@kroeckx Yes we can merge this. Please follow-up with conflicts and we'll proceed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: pending response Pending response from the issue/PR opener
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants