Skip to content

Conversation

@jullanggit
Copy link
Contributor

Adds comments to the documentation for distributed compilation explaining the requirement of the token being a valid HTTP header value.
Hopefully this saves the next person running into this a bit of time :)

@jullanggit jullanggit changed the title Add comments about auth token requirements docs: add comments about auth token requirements Feb 1, 2026
@sylvestre sylvestre merged commit fafcea5 into mozilla:main Feb 2, 2026
43 checks passed
@sylvestre
Copy link
Collaborator

thanks

@codecov-commenter
Copy link

codecov-commenter commented Feb 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.04%. Comparing base (9f6cba4) to head (df698fb).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2583      +/-   ##
==========================================
+ Coverage   72.03%   72.04%   +0.01%     
==========================================
  Files          66       66              
  Lines       36953    36953              
==========================================
+ Hits        26619    26623       +4     
+ Misses      10334    10330       -4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jullanggit jullanggit deleted the auth-token-reqs branch February 2, 2026 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants