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

Update docs to use https #3652

Closed
wants to merge 1 commit into from
Closed

Conversation

MikeLund
Copy link
Contributor

@MikeLund MikeLund commented Oct 5, 2016

Simple change to resolve #3651

@wweich
Copy link
Member

wweich commented Oct 5, 2016

@st-review merge it

jenkins: use https when downloading docs (fixes #3651)

@st-review
Copy link

👌 Merged as 1e54a3e. Thanks, @MikeLund!

@st-review st-review closed this Oct 5, 2016
st-review pushed a commit that referenced this pull request Oct 5, 2016
@fheinle
Copy link

fheinle commented Oct 5, 2016

It works. As a side note: passing -L to curl will follow HTTP redirects to their new destination, allowing for more flexibility on your webserver's side.

@wweich
Copy link
Member

wweich commented Oct 5, 2016

As we are all very security conscious, the servers will keep https (as long as there are free certs from Let's Encrypt or an alternative) and using https directly is imho the way to go for now.

@canton7
Copy link
Member

canton7 commented Oct 5, 2016

Sure, but I think that passing -L (as well as requesting https) can only do good.

@st-review st-review added the frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion label Nov 15, 2017
@syncthing syncthing locked and limited conversation to collaborators Nov 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Documentation in .deb package has HTML content in PDF files
5 participants