Skip to content

Change the verify mode to VERIFY_PEER#4

Open
fillyfuxilly wants to merge 4 commits intophuesler:masterfrom
tolingo:feature/verify-by-peer
Open

Change the verify mode to VERIFY_PEER#4
fillyfuxilly wants to merge 4 commits intophuesler:masterfrom
tolingo:feature/verify-by-peer

Conversation

@fillyfuxilly
Copy link
Copy Markdown

@fillyfuxilly fillyfuxilly commented Apr 16, 2020

This at least declines self-signed certificates and certificates for a wrong host. With this we add some security against man in the middle attacks.


Edit: The build failed because of the ruby versions
Travis changed the Ubuntu version and with it the ruby versions it supports. The versions 1.9.3 to 2.1.0 fail because they are only compatible with rake <11. Version 2.1.2 is not supported at all anymore.

I tested if a "newer" version(2.1.5) would work, but it also has trouble with the rake version, so I will try to set the rake version to be < 11.

Edit 2: Solution
It works with specifying a rake version <11, but the ruby version 2.1.2 still doesn't get supported by Travis' current Ubuntu version. See this list.
Replacing it with newer versions probably makes sense.

Wendy F added 2 commits April 16, 2020 10:27
This at least declines self-signed certificates and certificates for a wrong host.
With versions 1.9.3 to 2.1.0 it won't find the method 'last_comment' and version 2.1.2 is not supported by Ubuntu 16.04.6 that Travis currently uses for this project.
@fillyfuxilly fillyfuxilly changed the title Change the verify mode to VERIFY_PEER [WIP] Change the verify mode to VERIFY_PEER Apr 16, 2020
Wendy F added 2 commits April 16, 2020 12:29
Remove v2.1.2, because it doesn't get supported.
Run travis with newer versions instead.
@fillyfuxilly fillyfuxilly changed the title [WIP] Change the verify mode to VERIFY_PEER Change the verify mode to VERIFY_PEER Apr 16, 2020
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.

1 participant