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

ATH::Listeners::CORS improvements #265

Merged
merged 5 commits into from
Feb 17, 2023
Merged

ATH::Listeners::CORS improvements #265

merged 5 commits into from
Feb 17, 2023

Conversation

Blacksmoke16
Copy link
Member

@Blacksmoke16 Blacksmoke16 commented Feb 17, 2023

  • Add debug logging to help with figuring out issues with setting up CORS
  • Refactor listener to be a bit simpler, essentially reverts part of 5035a90
  • Manually install pkg-config before running tests on mac to workaround libxml2 not found on latest macos-12 actions/runner-images#7128
  • Skip installing executable when running tests via CI, saves like 30s/run

Resolves #263

Refactor listener to be a bit simpler, essentially reverts part of 5035a90
Lower severity to `TRACE`
@Blacksmoke16 Blacksmoke16 added component:framework kind:enhancement New functionality to an existing feature labels Feb 17, 2023
@Blacksmoke16 Blacksmoke16 merged commit 32521d7 into master Feb 17, 2023
@Blacksmoke16 Blacksmoke16 deleted the cors-logging branch February 17, 2023 02:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

Add debug logging to ATH::Listeners::CORS
1 participant