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 UPnP Support #1036

Open
SergioDemianLerner opened this issue Oct 22, 2019 · 7 comments
Open

Add UPnP Support #1036

SergioDemianLerner opened this issue Oct 22, 2019 · 7 comments
Labels
Gitcoin hackathon See https://gitcoin.co/hackathon/web3-world/

Comments

@SergioDemianLerner
Copy link
Contributor

SergioDemianLerner commented Oct 22, 2019

Prize Bounty

The Prize Bounty is 12.25 ETH

Challenge Description

The peer discovery component has a problem when announcing the local node to another peers: it doesn't traverse through NAT routers.
A way to address this would be implementing UPnP protocol.

https://en.wikipedia.org/wiki/Universal_Plug_and_Play

An sample solution applied to Pegasys Phanteon is this: PegaSysEng/pantheon#1334

Submission Requirements

A valid submission should modify the RSKj node to allow seamless discovery and communication through NAT routers using uPnP.
Submissions should not require large external library dependencies unless this is proven necessary.

The code should respect the contribution standards specified in https://github.com/rsksmart/rskj/blob/master/CONTRIBUTING.md

Submission Deadline

The deadline for submissions is November 11, 2019.

Judging Criteria

The prize will be awarded to the first valid submission.

Winner Announcement Date

The submission will be reviewed between November 11 and November 18, 2019. Winners will be privately notified before a public announcement of all hackathon prices given by IOV Labs. The announcement and prize payouts will occur before November 31, 2019.

@SergioDemianLerner SergioDemianLerner added the Gitcoin hackathon See https://gitcoin.co/hackathon/web3-world/ label Oct 22, 2019
@SergioDemianLerner SergioDemianLerner changed the title Add UPnP Support (Gitcoin hackaton project sponsored by IOV Labs) Add UPnP Support Oct 22, 2019
@gitcoinbot
Copy link

gitcoinbot commented Oct 31, 2019

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work has been started.

These users each claimed they can complete the work by 2 weeks, 4 days ago.
Please review their action plans below:

1) coejoder has started work.

Will integrate UPnP into peer discovery, using acceptance criteria similar to linked sample solution, as applicable to RSKj.

Learn more on the Gitcoin Issue Details page.

@CoeJoder
Copy link

CoeJoder commented Oct 31, 2019

@SergioDemianLerner Quick question regarding this bounty: would you accept a submission which contained some code licensed under Apache v2? I ask because the UpnpNatManager of the Besu project (formerly Pegasys Pantheon) is a useful abstraction here with minimal changes. It would require a copy of the Apache v2 license to be included in the source and distribution, as well as a comment in the code notifying that modifications have been made.
See: https://www.apache.org/licenses/GPL-compatibility.html

@CoeJoder
Copy link

CoeJoder commented Nov 1, 2019

Didn't get an answer and the clock is ticking, so I've found a different UPnP lib which is LGPL (weUPnP). It also pulls in less dependencies, so that's a plus.

@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work for 12.502 ETH (2373.32 USD @ $189.83/ETH) has been submitted by:

  1. @coejoder

@SergioDemianLerner please take a look at the submitted work:


CoeJoder added a commit to CoeJoder/rskj that referenced this issue Nov 8, 2019
CoeJoder added a commit to CoeJoder/rskj that referenced this issue Nov 8, 2019
CoeJoder added a commit to CoeJoder/rskj that referenced this issue Nov 8, 2019
@CoeJoder
Copy link

@SergioDemianLerner Gitcoin just contacted me that the bounty is expiring in 2 days and that I should remind you to review the PR :) Btw the PR is here: #1088

CoeJoder added a commit to CoeJoder/rskj that referenced this issue Dec 12, 2019
nacho692 added a commit that referenced this issue Dec 13, 2019
Add UPnP support for peer discovery. Resolves #1036
@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


The funding of 12.502 ETH (1807.09 USD @ $144.54/ETH) attached to this issue has been approved & issued to @CoeJoder.

@hdiass
Copy link

hdiass commented Mar 18, 2021

whats the status on this ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Gitcoin hackathon See https://gitcoin.co/hackathon/web3-world/
Projects
None yet
Development

No branches or pull requests

4 participants