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.

Sign up for GitHub

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

Jump to bottom

Prefer peer with the highest reputation if multiple PoT proofs were gossiped #2320

Merged
merged 1 commit into from
Dec 13, 2023
Merged

Prefer peer with the highest reputation if multiple PoT proofs were gossiped #2320

merged 1 commit into from
Dec 13, 2023

Conversation

nazar-pc
Copy link
Member

Resolves #2142 by implementing corresponding update to PoT specification

Code contributor checklist:

@nazar-pc nazar-pc added the need to audit This change needs to be audited label Dec 13, 2023
@nazar-pc nazar-pc requested a review from rg3l3dr as a code owner December 13, 2023 09:43
@nazar-pc nazar-pc mentioned this pull request Dec 13, 2023
Closed
@nazar-pc nazar-pc enabled auto-merge December 13, 2023 13:29
@nazar-pc nazar-pc added this pull request to the merge queue Dec 13, 2023
Merged via the queue into main with commit bc81cca Dec 13, 2023
10 checks passed
@nazar-pc nazar-pc deleted the pot-gossip-priority branch December 13, 2023 15:16
@nazar-pc
Copy link
Member Author

Follow-up to this PR: #2325

@vanhauser-thc
Copy link
Collaborator

LGTM

@vanhauser-thc vanhauser-thc added audited This change was audited and removed need to audit This change needs to be audited labels Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vedhavyas vedhavyas vedhavyas approved these changes

@shamil-gadelshin shamil-gadelshin Awaiting requested review from shamil-gadelshin

@dariolina dariolina Awaiting requested review from dariolina

@rahulksnv rahulksnv Awaiting requested review from rahulksnv

@rg3l3dr rg3l3dr Awaiting requested review from rg3l3dr

Assignees
No one assigned
Labels
audited This change was audited
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Burst Denial of Service in Proof of Time
3 participants
@nazar-pc @vanhauser-thc @vedhavyas