Skip to content
This repository was archived by the owner on Jan 17, 2025. It is now read-only.

Add PyJWT to support IBM Certificate Manager#90

Closed
osenbach wants to merge 2 commits intoibm-functions:masterfrom
osenbach:master
Closed

Add PyJWT to support IBM Certificate Manager#90
osenbach wants to merge 2 commits intoibm-functions:masterfrom
osenbach:master

Conversation

@osenbach
Copy link

I'd like to use IBM Functions to support all of the various IBM Certificate Manager workflows, such as ordering, renewing, expiring, etc... and to do so I need to decode a JWT encoded payload send in the notification. Currently, this isn't possible via Python3 actions as a result of the PyJWT module not included as a requirement.

I'd like to use IBM Functions to support all of the various IBM Certificate Manager workflows, such as ordering, renewing, expiring, etc... and to do so I need to decode a JWT encoded payload send in the notification.  Currently, this isn't possible via Python3 actions as a result of the PyJWT module not included as a requirement.
@osenbach
Copy link
Author

@dubee or @falkzoll mind a review / merge?

Same change as 3.7, but to 3.6 for alignment
@osenbach
Copy link
Author

@falkzoll any chance we can add PyJWT so that I can decode IBM Cert Manager messages?

@falkzoll
Copy link
Contributor

falkzoll commented Aug 1, 2019

Sorry did not see this PR at the first time :-(. Let me have a look to see if we can add this.
Would it be ok for you if we would only add it to the latest (python3.7) runtime?

@osenbach
Copy link
Author

Hey @falkzoll. Sure!

@falkzoll
Copy link
Contributor

Added PyJWT to the python:3.7 runtime with PR #95. Thanks for providing your input 👍 .

The updated version of this runtime is now available in IBM Cloud. Feel free to check it out and get back with more proposals to improve this runtime.
Closing this PR :-).

@falkzoll falkzoll closed this Oct 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants