mirror of
https://github.com/servo/servo.git
synced 2025-08-03 04:30:10 +01:00
Auto merge of #25768 - pshaughn:corsstar, r=jdm
Handle access-control header wildcards <!-- Please describe your changes on the following line: --> We were checking Access-Control-Expose-Headers for wildcards inconsistently and then discarding the result; this fixes the check and its use, passing the WPT test for having a wildcard there. --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #24913 <!-- Either: --> - [X] There are tests for these changes OR - [ ] These changes do not require tests because ___ <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
This commit is contained in:
commit
795dab71ff
3 changed files with 13 additions and 29 deletions
|
@ -340,15 +340,16 @@ pub fn main_fetch(
|
|||
.map(|v| v.iter().collect());
|
||||
match header_names {
|
||||
// Subsubstep 2.
|
||||
Some(ref list) if request.credentials_mode != CredentialsMode::Include => {
|
||||
if list.len() == 1 && list[0] == "*" {
|
||||
response.cors_exposed_header_name_list = response
|
||||
.headers
|
||||
.iter()
|
||||
.map(|(name, _)| name.as_str().to_owned())
|
||||
.collect();
|
||||
}
|
||||
},
|
||||
Some(ref list)
|
||||
if request.credentials_mode != CredentialsMode::Include &&
|
||||
list.iter().any(|header| header == "*") =>
|
||||
{
|
||||
response.cors_exposed_header_name_list = response
|
||||
.headers
|
||||
.iter()
|
||||
.map(|(name, _)| name.as_str().to_owned())
|
||||
.collect();
|
||||
}
|
||||
// Subsubstep 3.
|
||||
Some(list) => {
|
||||
response.cors_exposed_header_name_list =
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue