Addresses an issue with certificate validation #2661

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@atavistock

This addresses wayneeseguin#2640

Its handled in the same way that this issue appears to be handled at another place in the existing RVM source (the message was simply lifted from there).

Remove insecure curl flag for file_exists_at_url method and provide s…
…ame certificate error message as seen elsewhere in RVM
@mpapis

This comment has been minimized.

Show comment Hide comment
@mpapis

mpapis Mar 1, 2014

Owner

as discussed in #2640 we can not print warnings, this function is supposed to check if remote file exist, no matter if certificates are valid or not, proper behavior would be to run first with --insecure and only retry without the flag when there is error 60 (this is faster way for most of the users).

Owner

mpapis commented Mar 1, 2014

as discussed in #2640 we can not print warnings, this function is supposed to check if remote file exist, no matter if certificates are valid or not, proper behavior would be to run first with --insecure and only retry without the flag when there is error 60 (this is faster way for most of the users).

@mpapis mpapis closed this in 613c940 Mar 7, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment