Skip to content
This repository has been archived by the owner on Sep 19, 2021. It is now read-only.

fix shields.io badge download #69

Merged
merged 1 commit into from
Sep 12, 2015
Merged

fix shields.io badge download #69

merged 1 commit into from
Sep 12, 2015

Conversation

justinmk
Copy link
Member

https://bugs.launchpad.net/ubuntu/+source/openssl/+bug/861137

neovim/neovim#3308 (comment)

badges/shields#439 (comment)

To test this, there isn't much we can do except commit it. The wget invocations works fine on my local ubuntu VM, so there's some difference between the version of ubuntu that travis is running.

justinmk added a commit that referenced this pull request Sep 12, 2015
@justinmk justinmk merged commit b316f40 into neovim:master Sep 12, 2015
@justinmk justinmk deleted the fixbadgedownload branch September 12, 2015 22:03
@justinmk
Copy link
Member Author

That didn't work, different issue now:

--2015-09-12 22:11:03--  https://img.shields.io/badge/clang_analysis-0-00ff00.svg
Resolving img.shields.io (img.shields.io)... 104.18.45.160, 104.18.44.160
Connecting to img.shields.io (img.shields.io)|104.18.45.160|:443... connected.
OpenSSL: error:14094410:SSL routines:SSL3_READ_BYTES:sslv3 alert handshake failure
Unable to establish SSL connection.

Trying --secure-protocol=TLSv1 now.

justinmk added a commit that referenced this pull request Sep 12, 2015
@justinmk
Copy link
Member Author

That also failed.

--2015-09-12 22:26:00--  https://img.shields.io/badge/clint-0-green.svg
Resolving img.shields.io (img.shields.io)... 104.18.45.160, 104.18.44.160
Connecting to img.shields.io (img.shields.io)|104.18.45.160|:443... connected.
OpenSSL: error:14094438:SSL routines:SSL3_READ_BYTES:tlsv1 alert internal error
Unable to establish SSL connection.

Falling back to http://.

justinmk added a commit that referenced this pull request Sep 12, 2015
justinmk added a commit that referenced this pull request Jan 4, 2016
- Previous attempt via --sslv3 failed because SSLv3 (nor v2) is not
  supported by the origins.
- Previous attempt via --tlsv1.2 failed because that option is not
  available in the `curl` provided by Travis.

References #69
References #70
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant