<html><head><style>body{font-family:Helvetica,Arial;font-size:13px}</style></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Hi Erik,</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Thank you for your contribution! Some of your work overlaps with the interfaces we're currently building (https://github.com/pyca/cryptography/pull/1906/), but there is implementation work and other interfaces that will be very useful. Incidentally, your approach to fullname/relativename is one of the discussions underway on that PR right now. </div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">The normal way we do contribution and code review is via pull requests on GitHub. You can put your initial work up for discussion by doing the following:</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">* Fork the project to your own account on github and check it out</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">* Branch, commit, and push the branch to your own repository</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">* Open a pull request against pyca/cryptography by following these instructions (https://help.github.com/articles/creating-a-pull-request/)</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">We're also available on freenode in #cryptography-dev to chat any time.</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">I personally would say this work should probably wait on merging #1906, at which point we can pull in the CRLDistributionPoints OpenSSL implementation, then follow that up with a PR for the CRL object, and finally the OpenSSL implementation of a parser to build the CRL object.</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">-Paul</div> <div id="bloop_sign_1431023179708195840" class="bloop_sign"><div style="font-family:helvetica,arial;font-size:13px"><br></div></div> <br><p style="color:#000;">On May 7, 2015 at 12:01:15 PM, Erik Trauschke (<a href="mailto:erik.trauschke@gmail.com">erik.trauschke@gmail.com</a>) wrote:</p> <blockquote type="cite" class="clean_bq"><span><div><div></div><div>Hi all,

<br>

<br>For my project I need cryptography to support basic handling of CRLs,

<br>revoked certificates and CRLDistributionPoints.

<br>

<br>I attached a patch which adds these interfaces, including test cases

<br>for it. I have never provided patches to a github project so I'm not

<br>sure how the process works.

<br>Do you have a separate place for code reviews (I haven't seen code

<br>review discussions on this list)?

<br>

<br>I would appreciate if someone could look at my changes and put them

<br>back to the gate, but let me know if I need to approach this

<br>differently.

<br>

<br>Thanks

<br>Erik

<br><hr>_______________________________________________

<br>Cryptography-dev mailing list

<br>Cryptography-dev@python.org

<br>https://mail.python.org/mailman/listinfo/cryptography-dev

<br></div></div></span></blockquote></body></html>