Skip to content
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

feat: add authentication options to hooks #3633

Merged
merged 6 commits into from
Sep 25, 2023
Merged

feat: add authentication options to hooks #3633

merged 6 commits into from
Sep 25, 2023

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%> (ø)

driver/config/provider.go Outdated Show resolved Hide resolved
@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
29 checks passed
@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