Skip to content

feat: add authentication options to hooks#3633

Merged
aeneasr merged 6 commits intomasterfrom
feat/webhook-auth
Sep 25, 2023
Merged

feat: add authentication options to hooks#3633
aeneasr merged 6 commits intomasterfrom
feat/webhook-auth

Conversation

@zepatrik
Copy link
Member

Pre-requisite for ory/network#324

@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Merging #3633 (ec7ffc0) into master (3615e3d) will decrease coverage by 0.07%.
The diff coverage is 71.42%.

❗ Current head ec7ffc0 differs from pull request most recent head af5ec08. Consider uploading reports for the commit af5ec08 to get more accurate results

@@            Coverage Diff             @@
##           master    #3633      +/-   ##
==========================================
- Coverage   76.08%   76.02%   -0.07%     
==========================================
  Files         132      132              
  Lines        9991    10015      +24     
==========================================
+ Hits         7602     7614      +12     
- Misses       1867     1877      +10     
- Partials      522      524       +2     
Files Changed Coverage Δ
oauth2/token_hook.go 66.66% <61.29%> (-5.13%) ⬇️
driver/config/provider.go 79.19% <81.81%> (ø)
oauth2/refresh_hook.go 67.64% <100.00%> (ø)

@zepatrik zepatrik requested a review from aeneasr September 22, 2023 11:40
@zepatrik zepatrik changed the title feat: allow setting HTTP headers in hooks feat: add authentication options to hooks Sep 22, 2023
@aeneasr aeneasr merged commit 5c8e792 into master Sep 25, 2023
@aeneasr aeneasr deleted the feat/webhook-auth branch September 25, 2023 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants