-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds Generated Ruby Client Example #42
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@john-odonnell Great work on this! I left some comments but it seems pretty straightforward good addition!
9a94331
to
e574030
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@john-odonnell Code is looking great - just a few more comments!
b2daa60
to
b2a637b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good! I think most of the remaining work should be focused on getting the example to run inside a container.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
b2a637b
to
ee2e328
Compare
d166eb4
to
bf780f2
Compare
bf780f2
to
7270568
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really good improvements! Just a few more things to polish up, and it'll be ready to go.
55f4965
to
4bd77d3
Compare
Adds localhost as a Subject Alternative Name on leafnode certificates. Allows SSL verification on local Conjur instances. Updates Python client demo to use HTTPS.
Previously, the policies were defined in the example. Moving their definition to independent files will allow multiple examples to use the same policies in their workflows.
Demonstrates the same API endpoints as the Python example: - authentication - password change - policy load - API key rotation of foreign role - secret storage and retrieval
4bd77d3
to
39ee1cd
Compare
81d542b
to
3ab960d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost there! Just two small things
Adds unique READMEs to each example, and generalizes terminology in project main README regarding examples. Updates CHANGELOG.
Python exits on exceptions. Removing these blocks simplify the example, and provide more extensive error messages if an exception does occur.
Done to eliminate the need for a user to install RVM before using the example. This change includes the following: - Adds a ruby-example service to the project's docker-compose - Adds Dockerfile for building the ruby-example service - Updates to the Ruby example program and start script to account for these changes - Updates to start_conjur, integration_tests, and GHActions integration test workflow - These processes now build only the docker-compose services they require - GHAction workflow uses integration_tests script, reducing duplicate code - Documentation regarding the examples are updated across the project
Script originally discarded output from other scripts, like generate_client and start_conjur. Redirection was limited to stdout, allowing stderr to display.
3ab960d
to
875dadb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM ✨
What does this PR do?
This PR adds an example which uses a spec-generated Ruby client.
The example is similar in workflow to the existing Python client example.
Updates certificates to include
localhost
as a leafnode SAN, allows SSL verification againstlocal Conjur instances.
What ticket does this PR close?
Closes #12
Checklists
Change log
Test coverage
Documentation
README
s) were updated in this PR, and/or there is a follow-on issue to update docs, or