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

Require auth for all buildbot actions #272

Merged
merged 1 commit into from Mar 24, 2016
Merged

Require auth for all buildbot actions #272

merged 1 commit into from Mar 24, 2016

Conversation

@edunham
Copy link
Contributor

edunham commented Mar 23, 2016

@edunham
Copy link
Contributor Author

edunham commented Mar 23, 2016

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Mar 23, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Mar 23, 2016

📌 Commit c152695 has been approved by larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented Mar 23, 2016

Testing commit c152695 with merge 57aa7a0...

bors-servo added a commit that referenced this pull request Mar 23, 2016
Require auth for all buildbot actions

http://docs.buildbot.net/0.8.1/WebStatus-Configuration-Parameters.html

Addresses #172

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/272)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Mar 23, 2016

💔 Test failed - travis

@edunham
Copy link
Contributor Author

edunham commented Mar 24, 2016

Tests failed only due to the known apt issues in Travis.

@edunham edunham merged commit a6be748 into servo:master Mar 24, 2016
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
homu Test failed
Details
@edunham edunham deleted the edunham:auth branch Mar 24, 2016
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.

None yet

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