[Nfd-dev] question about nrd

Alex Afanasyev alexander.afanasyev at ucla.edu
Wed Mar 12 17:57:37 PDT 2014


Ok. Let's keep separate... But what should I do with the outstanding commits?  Approve and submit without tests and security?  Wait for tests and/or security?

There are other things like a broken logic---right now NRD will always return status 200 success, even if command to NFD will fail.  Should this be fixed before the submission?

---
Alex

On Mar 12, 2014, at 5:30 PM, Junxiao Shi <shijunxiao at email.arizona.edu> wrote:

> Logging: trivial code.
> Command Interest verification: NRD needs more complex logic such as per-namespace authorization.
> Configuration parser: different file format.
> 
> If NFD and NRD should be in the same repository because these three modules are shared, same logic would cause NLSR to go into the same repository.
> 
> I prefer to keep them separate, because they are different programs.
> 
> Yours, Junxiao
> 
> 
> 
> Alex Afanasyev <alexander.afanasyev at ucla.edu> wrote:
> I did several iterations of updates in nrd repo, but I still have a problem with approving 2 outstanding commits.   Neither of them has unit tests, but my primary problem is lack of command interest verification, which exists now in NFD.  It doesn't make sense for me to be extremely strict in NFD and yet allow everybody to register prefixes using NRD...
> 
> Also. After looking more into the code I'm less and less convinced that NRD should be in a separate repository.  A lot of things have been already implemented in NFD (logging, command interest verification, config file) and having separate repo basically means that the code needs to be copied (and maintained) in two places, instead of one.
> 
> How should we proceed?
> 
> ---
> Alex
> 
> 
> 
> Nfd-dev mailing list
> Nfd-dev at lists.cs.ucla.edu
> http://www.lists.cs.ucla.edu/mailman/listinfo/nfd-dev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.lists.cs.ucla.edu/pipermail/nfd-dev/attachments/20140312/e095e9c6/attachment.html>


More information about the Nfd-dev mailing list