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

Switch to https://build.servo.org #906

Closed
wants to merge 2 commits into from
Closed

Conversation

@Darkspirit
Copy link

Darkspirit commented Oct 31, 2018

@jdm
I don't want to just ping you, so I make this pull request.
This should help with #423 (comment), but is not tested with Vagrant.

  • A next step would be to change it to a permanent http-to-https redirection and to activate HSTS.
  • Current TLS config is bad, therefore only modern ciphersuites should be allowed.
  • http://build.servo.org:54856/travis might already not exist anymore?
  • Please feel free to edit this pull request.

(Background: servo.org enforces HSTS since bug 1482947 and is ready to be HSTS-preloaded.)


This change is Reviewable

Darkspirit added 2 commits Oct 31, 2018
@Darkspirit
Copy link
Author

Darkspirit commented Nov 6, 2018

@jdm
There is a long list of travis webhooks trying to connect to http://build.servo.org:54856/travis which is already 404 Not Found when accessing with curl.
Since #454 Homu is checking build statuses directly at GitHub

saltfs/homu/files/cfg.toml

Lines 268 to 271 in 5dcb05a

{% if repo.get('travis', True) %}
[repo."{{ name }}".status.travis]
context = 'continuous-integration/travis-ci/push'
{% endif %}
and doesn't rely on Travis webhook notifications, if I'm not wrong:
"travis": False,

Shall I create PRs removing each travis webhook? Or do you want to ignore them?

@jdm
Copy link
Member

jdm commented Nov 6, 2018

Removing those unneeded webhook notifications would be useful.

@Darkspirit
Copy link
Author

Darkspirit commented Nov 6, 2018

I'll split this to multiple PRs as well.

@Darkspirit Darkspirit closed this Nov 6, 2018
bors-servo added a commit to servo/download.servo.org that referenced this pull request Nov 7, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

Please also edit the Website URL at the top of https://github.com/servo/download.servo.org to https.
bors-servo added a commit to servo/rust-azure that referenced this pull request Nov 7, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-azure/295)
<!-- Reviewable:end -->
bors-servo added a commit to servo/core-foundation-rs that referenced this pull request Nov 7, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/core-foundation-rs/271)
<!-- Reviewable:end -->
bors-servo added a commit to servo/rust-url that referenced this pull request Nov 7, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-url/470)
<!-- Reviewable:end -->
bors-servo added a commit to servo/mozjs that referenced this pull request Nov 7, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/mozjs/160)
<!-- Reviewable:end -->
bors-servo added a commit to servo/unicode-bidi that referenced this pull request Nov 7, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/unicode-bidi/50)
<!-- Reviewable:end -->
bors-servo added a commit to servo/rust-mozjs that referenced this pull request Nov 7, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-mozjs/446)
<!-- Reviewable:end -->
bors-servo added a commit to servo/rust-smallvec that referenced this pull request Nov 7, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-smallvec/129)
<!-- Reviewable:end -->
bors-servo added a commit to servo/rust-mozjs that referenced this pull request Nov 7, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-mozjs/446)
<!-- Reviewable:end -->
bors-servo added a commit to servo/heapsize that referenced this pull request Nov 7, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/heapsize/102)
<!-- Reviewable:end -->
bors-servo added a commit to servo/unicode-script that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)
bors-servo added a commit to servo/cgl-rs that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/cgl-rs/27)
<!-- Reviewable:end -->
bors-servo added a commit to servo/devices that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/devices/31)
<!-- Reviewable:end -->
bors-servo added a commit to servo/fontsan that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/fontsan/23)
<!-- Reviewable:end -->
bors-servo added a commit to servo/futf that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/futf/9)
<!-- Reviewable:end -->
bors-servo added a commit to servo/gaol that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)
bors-servo added a commit to servo/gleam that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/gleam/177)
<!-- Reviewable:end -->
bors-servo added a commit to servo/libexpat that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/libexpat/24)
<!-- Reviewable:end -->
bors-servo added a commit to servo/libfontconfig that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/libfontconfig/40)
<!-- Reviewable:end -->
bors-servo added a commit to servo/libfreetype2 that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/libfreetype2/27)
<!-- Reviewable:end -->
bors-servo added a commit to servo/html5ever that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)
bors-servo added a commit to servo/mozangle that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)
bors-servo added a commit to servo/rust-harfbuzz that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-harfbuzz/125)
<!-- Reviewable:end -->
bors-servo added a commit to servo/rust-cssparser that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-cssparser/233)
<!-- Reviewable:end -->
bors-servo added a commit to servo/mozangle that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)
bors-servo added a commit to servo/rust-fontconfig that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-fontconfig/36)
<!-- Reviewable:end -->
bors-servo added a commit to servo/rust-fnv that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-fnv/19)
<!-- Reviewable:end -->
bors-servo added a commit to servo/rust-freetype that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-freetype/56)
<!-- Reviewable:end -->
bors-servo added a commit to servo/rust-glx that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-glx/22)
<!-- Reviewable:end -->
bors-servo added a commit to servo/rust-layers that referenced this pull request Nov 10, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-layers/246)
<!-- Reviewable:end -->
bors-servo added a commit to servo/rust-fontconfig that referenced this pull request Nov 13, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-fontconfig/36)
<!-- Reviewable:end -->
bors-servo added a commit to servo/app_units that referenced this pull request Nov 13, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)
bors-servo added a commit to servo/servo-starters that referenced this pull request Nov 13, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)
bors-servo added a commit to servo/string-cache that referenced this pull request Nov 13, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/string-cache/213)
<!-- Reviewable:end -->
bors-servo added a commit to servo/rust-stb-image that referenced this pull request Nov 13, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-stb-image/92)
<!-- Reviewable:end -->
bors-servo added a commit to servo/highfive that referenced this pull request Nov 13, 2018
Remove unneeded webhook notification

servo/saltfs#906 (comment)
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.