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

Specify base64url encoding for auction result header #631

Merged
merged 1 commit into from
Jul 27, 2023

Conversation

brusshamilton
Copy link
Contributor

Specify the encoding for the auction response and clarify support multiple auction response headers.

aarongable pushed a commit to chromium/chromium that referenced this pull request Jun 16, 2023
Sites using the Ad-Auction-Result header to provide the result from
ad auctions may want to provide the results from multiple auctions
in a single response. This CL adds support for that and also changes
the encoding from base16 to base64url to reduce the header overhead.

This implements the changes described in the explainer pull request: WICG/turtledove#631

Bug: 1442274
Change-Id: I046d36aac46b40119a3129adffae26e623e2c6b8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4614132
Reviewed-by: Yao Xiao <[email protected]>
Commit-Queue: Russ Hamilton <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1158971}
@JensenPaul JensenPaul added the Non-breaking Feature Request Feature request for functionality unlikely to break backwards compatibility label Jun 23, 2023
@JensenPaul JensenPaul merged commit 278a445 into WICG:main Jul 27, 2023
1 check passed
github-actions bot added a commit that referenced this pull request Jul 27, 2023
SHA: 278a445
Reason: push, by JensenPaul

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@brusshamilton brusshamilton deleted the specifyEncoding branch August 4, 2023 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Non-breaking Feature Request Feature request for functionality unlikely to break backwards compatibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants