Re: CertificateFactory does not conforms to JAVA specifications
Massimiliano Ziccardi <[email protected]> Mon, 8 Apr 2019 11:45:58 +0200
| Newsgroups | gmane.comp.encryption.bouncy-castle.devel |
|---|---|
| Message-ID | <CAPwY8YAV6PQUN0e=5BJjzHaoRGtjn3pO_n6rtE-6H9EF+h-jQw@mail.gmail.com> |
Hi David, I see a comment from you here https://github.com/bcgit/bc-java/pull/459 saying that it has been merged, however I see the `PR` as closed (instead of merged) and searching through the repo I don't see any of the PR's changes into `master`. Cheers, Massimiliano Il giorno sab 16 mar 2019 alle ore 20:59 David Hook <dgh-rTAZ0PM/[email protected]> ha scritto: > > Hi Massimiliano, > > It's on my list, it will be probably be another couple of weeks before I > get to it though (there's a couple of other things on the list I have to > still deal with). > > Apologies, > > David > > On 15/3/19 11:26 pm, Massimiliano Ziccardi wrote: > > Hello guys! > > About one month ago, I opened this issue: > > - https://github.com/bcgit/bc-java/issues/457 - CertificateFactory > does not conforms to specifications > > And I created a PR that should solve/mitigate the issue: > > - https://github.com/bcgit/bc-java/pull/459 > > But I never received any comment. > > We already discussed this issue some year ago here > <http://bouncy-castle.1462172.n4.nabble.com/Problem-in-the-bouncycastle-CertificateFactory-implementation-td4655904.html> > > I really believe it should be fixed. As it is, it creates incoherent > results when passing from any compliant provider (Sun, IAIK, etc) to > BouncyCastle. > > Code relying to the fact that bad input should raise an exception, could > start failing with a NullPointerException if changing the provider to > BouncyCastle. > > WDYT? > > Regards, > Massimiliano > > >