FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat(Utils): Add signature verification utility function by jhodges10 · Pull Request #81 · Frameio/python-frameio-client · GitHub

feat(Utils): Add signature verification utility function - #81

Open
jhodges10 wants to merge 3 commits into
developfrom
jh/add-signature-verification-util
Open

feat(Utils): Add signature verification utility function#81
jhodges10 wants to merge 3 commits into
developfrom
jh/add-signature-verification-util

Conversation

jhodges10 commented Jun 20, 2021
edited
Loading

Copy link
Copy Markdown
Contributor

DEVREL-558

What I'm doing

  • Adding signature verification util for custom actions/webhooks

Tests

  • Need to add test for this function

jhodges10 changed the title Add signature verification util WIP Add signature verification util Jun 20, 2021
jhodges10 added the enhancement New feature or request label Jun 20, 2021
jhodges10 requested a review from lezoudali July 7, 2021 09:18
jhodges10 force-pushed the jh/add-signature-verification-util branch from 4d883c6 to a74dbfd Compare July 30, 2021 04:18
jhodges10 changed the title WIP Add signature verification util feat(Utils): Add signature verification utility function Aug 3, 2021
jhodges10 mentioned this pull request Aug 3, 2021
Comment thread frameioclient/lib/utils.py Outdated

lezoudali left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM. as long as you've tested and it works as expected. I'd add some tests? 🤔

Co-authored-by: Lezou Dali <lezou.dali@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL