PR#88: Use TLS for test suite - - Pagure.io

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

ipsilon

#88 Use TLS for test suite

Merged May 4, 2016 by puiterwijk. Opened May 4, 2016 by puiterwijk.

puiterwijk/ipsilon testsuite-ssl into master

Download 88.patch

puiterwijk commented May 4, 2016

This patch makes the test suite setup and use TLS for all
of the test cases.

Signed-off-by: Patrick Uiterwijk puiterwijk@redhat.com

rebased

May 4, 2016

rebased

May 4, 2016

rcritten commented May 4, 2016

The patch looks ok and it works. I think I'd prefer for the cert stuff be broken out into separate functions, if only for readability. So add a generate_ca() and a generate_cert() call (or name it as you wish).

rebased

May 4, 2016

rebased

May 4, 2016

rcritten commented May 4, 2016

LGTM, all tests are passing, ACK.

rebased

May 4, 2016

Pull-Request has been merged by puiterwijk

May 4, 2016

Metadata

Assignee

None

Tags

No Tags

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.