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

[7.x] sourcemap: fix data race in test (#5842) #5855

Merged
merged 1 commit into from
Aug 3, 2021

Conversation

axw
Copy link
Member

@axw axw commented Aug 2, 2021

Backports the following commits to 7.x:

Fix data race in TestFleetFetch: as there are two
servers, one may update hasAuth concurrently with
the test checking the value.

Change the way the auth header is checked to be
similar to how the header would be checked in a
real fleet-server, and respond with 401 if the
auth header is invalid.
@axw axw added the backport label Aug 2, 2021
@axw axw enabled auto-merge (squash) August 2, 2021 09:14
@axw
Copy link
Member Author

axw commented Aug 2, 2021

run elasticsearch-ci/docs

@apmmachine
Copy link
Contributor

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-08-02T09:14:28.344+0000

  • Duration: 43 min 5 sec

  • Commit: 2c1bd46

Test stats 🧪

Test Results
Failed 0
Passed 4517
Skipped 15
Total 4532

Trends 🧪

Image of Build Times

Image of Tests

@axw axw disabled auto-merge August 3, 2021 01:22
@axw axw merged commit 68adbe5 into elastic:7.x Aug 3, 2021
@axw axw deleted the backport/7.x/pr-5842 branch August 3, 2021 01:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants