License of NLKBufferedReader

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

License of NLKBufferedReader

Peter Wicks (pwicks)
While digging into ReplaceText today, I noticed that org.apache.nifi.processors.standard.util.NLKBufferedReader is actually a direct copy of the JDK version of BufferedReader, with a few minor code edits made on top of it so that new line characters are kept. This code was in the original code commit from way back.

It feels like there is a licensing issue here, as this is Oracle source code. Joe?

--Peter

Reply | Threaded
Open this post in threaded view
|

Re: License of NLKBufferedReader

Joe Witt
Yeah that looks super sketchy frankly and we need to address that immediately.

JeffStorck: This is an RC blocker.  I'll file a JIRA shortly.  I am
shocked this hasn't been caught before.
On Tue, Oct 16, 2018 at 3:11 PM Peter Wicks (pwicks) <[hidden email]> wrote:
>
> While digging into ReplaceText today, I noticed that org.apache.nifi.processors.standard.util.NLKBufferedReader is actually a direct copy of the JDK version of BufferedReader, with a few minor code edits made on top of it so that new line characters are kept. This code was in the original code commit from way back.
>
> It feels like there is a licensing issue here, as this is Oracle source code. Joe?
>
> --Peter
>