close
The Wayback Machine - https://web.archive.org/web/20201114121134/https://github.com/letsencrypt/boulder/pull/4376
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

Log the Origin header. #4376

Merged
merged 1 commit into from Jul 31, 2019
Merged

Log the Origin header. #4376

merged 1 commit into from Jul 31, 2019

Conversation

@jsha
Copy link
Contributor

@jsha jsha commented Jul 31, 2019

XHR requests from web-based ACME clients provide the User-Agent
of the browser that initiated the request, but the hostname of the site
that originated the request is sent in the Origin header. This will let
us better analyze web-based ACME traffic.

Fixes #4370

XHR requests from web-based ACME clients provide the User-Agent
of the browser that initiated the request, but the hostname of the site
that originated the request is sent in the Origin header. This will let
us better analyze web-based ACME traffic.
@jsha jsha requested a review from letsencrypt/boulder-developers as a code owner Jul 31, 2019
@cpu
cpu approved these changes Jul 31, 2019
Copy link
Contributor

@cpu cpu left a comment

Looks good to me 👍

@rolandshoemaker rolandshoemaker merged commit c777dfe into master Jul 31, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@rolandshoemaker rolandshoemaker deleted the logorigin branch Jul 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants
You can’t perform that action at this time.