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: hot-reload Oauth2 CORS settings #3537

Merged
merged 3 commits into from
Jun 13, 2023
Merged

Conversation

zepatrik
Copy link
Member

@zepatrik zepatrik commented Jun 6, 2023

As client settings change over time, this should be fully hot-reloadable. Resolves regression of #1754 on Ory Network.

@zepatrik zepatrik requested a review from aeneasr as a code owner June 6, 2023 10:05
@codecov
Copy link

codecov bot commented Jun 6, 2023

Codecov Report

Merging #3537 (003cf3e) into master (fa8c053) will decrease coverage by 2.59%.
The diff coverage is 88.57%.

❗ Current head 003cf3e differs from pull request most recent head 12cc543. Consider uploading reports for the commit 12cc543 to get more accurate results

@@            Coverage Diff             @@
##           master    #3537      +/-   ##
==========================================
- Coverage   79.41%   76.83%   -2.59%     
==========================================
  Files         129      124       -5     
  Lines       11368     9108    -2260     
==========================================
- Hits         9028     6998    -2030     
+ Misses       1779     1668     -111     
+ Partials      561      442     -119     
Impacted Files Coverage Δ
driver/registry.go 24.24% <ø> (-9.10%) ⬇️
x/oauth2cors/cors.go 87.14% <87.87%> (-3.03%) ⬇️
driver/registry_base.go 85.98% <100.00%> (-2.01%) ⬇️

... and 26 files with indirect coverage changes

x/oauth2cors/cors.go Show resolved Hide resolved
x/oauth2cors/cors.go Show resolved Hide resolved
x/oauth2cors/cors.go Outdated Show resolved Hide resolved
@zepatrik zepatrik force-pushed the feat/hot-reload-oauth2-cors branch 2 times, most recently from 6474179 to d418ce0 Compare June 9, 2023 09:49
@zepatrik zepatrik force-pushed the feat/hot-reload-oauth2-cors branch from d418ce0 to 12cc543 Compare June 12, 2023 14:31
@aeneasr aeneasr merged commit a8ecf80 into master Jun 13, 2023
@aeneasr aeneasr deleted the feat/hot-reload-oauth2-cors branch June 13, 2023 11:45
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