> On May 14, 2015, 8:17 p.m., Andrew Stitcher wrote: > > proton-c/src/ssl/openssl.c, line 1263 > > <https://reviews.apache.org/r/34229/diff/1/?file=960166#file960166line1263> > > > > I think _oneline would be closer in format to using flags > > XN_FLAGS_ONELINE
Oops, I didn't mean to leave that in. The oneline approach was the first option, but it prints in slightly non-standard form (uses '/' as separator rather than ',') and on reading the docs its no longer recommended. > On May 14, 2015, 8:17 p.m., Andrew Stitcher wrote: > > proton-c/src/ssl/openssl.c, line 1265 > > <https://reviews.apache.org/r/34229/diff/1/?file=960166#file960166line1265> > > > > You probably need > > if (!subject) return NULL; > > here too (there may be circumstances in which we can get a certificate > > without a subject) Good point. - Gordon ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34229/#review83829 ----------------------------------------------------------- On May 14, 2015, 7:54 p.m., Gordon Sim wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34229/ > ----------------------------------------------------------- > > (Updated May 14, 2015, 7:54 p.m.) > > > Review request for qpid, Andrew Stitcher, Cliff Jansen, Kenneth Giusti, and > Rafael Schloming. > > > Bugs: PROTON-861 > https://issues.apache.org/jira/browse/PROTON-861 > > > Repository: qpid-proton-git > > > Description > ------- > > This is useful e.g. to determine whether a particular connection is > authorised for certain actions. > > The approach taken here is to expose the full subject as a string. It may be > that some subset of that is preferred, perhaps in a slightly different > format. However having the full subject is the simplest way to ensure that > everyone can get what they need, even if at the expense of a little string > manipulation. > > > Diffs > ----- > > proton-c/bindings/python/proton/__init__.py bc639e3 > proton-c/include/proton/ssl.h 0ac4aef > proton-c/src/ssl/openssl.c 2bbdda0 > > Diff: https://reviews.apache.org/r/34229/diff/ > > > Testing > ------- > > > Thanks, > > Gordon Sim > >
