Skip to content

Comments

[ADD] Initial version of OpenSSL container#1

Merged
tedsalmon merged 1 commit intomasterfrom
release/0.1
Jan 3, 2017
Merged

[ADD] Initial version of OpenSSL container#1
tedsalmon merged 1 commit intomasterfrom
release/0.1

Conversation

@lasley
Copy link
Member

@lasley lasley commented Jan 3, 2017

This brings over the OpenSSL exec from clouder-community/clouder#180 & adds tests.

cc @tedsalmon @YannickB

@lasley lasley self-assigned this Jan 3, 2017
@lasley lasley force-pushed the release/0.1 branch 4 times, most recently from 15f4050 to bf0a5b2 Compare January 3, 2017 07:20
* Dockerfile
* Entrypoint
* Parse_cert binary
Copy link
Contributor

@tedsalmon tedsalmon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍

# Copyright 2016 LasLabs Inc.
# License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl.html).

from __future__ import print_function
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For my own edification, apart from 3.x compat, what's the value add of importing the 3.x print function?

Copy link
Member Author

@lasley lasley Jan 3, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2/3.x compat

@tedsalmon tedsalmon merged commit 403a310 into master Jan 3, 2017
@tedsalmon tedsalmon deleted the release/0.1 branch January 3, 2017 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants