Closed superchilled closed 9 months ago
~WIP. Do not merge.~
Ready to merge
Merging #291 (2f2b46b) into main (e456fd3) will increase coverage by
0.01%
. The diff coverage is100.00%
.:exclamation: Current head 2f2b46b differs from pull request most recent head 34e8f9e. Consider uploading reports for the commit 34e8f9e to get more accurate results
@@ Coverage Diff @@
## main #291 +/- ##
==========================================
+ Coverage 97.93% 97.94% +0.01%
==========================================
Files 108 108
Lines 1988 1999 +11
==========================================
+ Hits 1947 1958 +11
Misses 41 41
Files | Coverage Δ | |
---|---|---|
lib/vonage/jwt.rb | 100.00% <100.00%> (ø) |
|
lib/vonage/messaging.rb | 100.00% <100.00%> (ø) |
|
lib/vonage/signature.rb | 100.00% <100.00%> (ø) |
|
lib/vonage/sms.rb | 100.00% <100.00%> (ø) |
|
lib/vonage/voice.rb | 100.00% <100.00%> (ø) |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
This PR adds functionality to the SDK to verify JWTs included in Vonage webhooks. Specifically it:
gemspec
dependency forvonage-jwt
to the new packageverify_hs256_signature
method to theVonage::JWT
classverify_webhook_token
wrapper methods toVoice
andMessaging
verify_webhook_sig
wrapper method toSMS
README
to explain the new functionality