Project

Profile

Help

Issue #5284

:verify is defined twice in ruby bindings

Added by jsherril@redhat.com about 1 year ago. Updated 6 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Katello
Sprint:
Sprint 57
Quarter:

Description

When loading my app, i see:

/home/vagrant/.gem/ruby/gems/pulpcore_client-3.0.0rc5.dev01565800523/lib/pulpcore_client/api_client.rb:51: warning: key :verify is duplicated and overwritten on line 52
/home/vagrant/.gem/ruby/gems/pulp_file_client-0.1.0b2.dev01565807323/lib/pulp_file_client/api_client.rb:51: warning: key :verify is duplicated and overwritten on line 52
/home/vagrant/.gem/ruby/gems/pulp_ansible_client-0.2.0b3.dev01565792570/lib/pulp_ansible_client/api_client.rb:51: warning: key :verify is duplicated and overwritten on line 52
/home/vagrant/.gem/ruby/gems/pulp_docker_client-4.0.0b6.dev01565789008/lib/pulp_docker_client/api_client.rb:51: warning: key :verify is duplicated and overwritten on line 52

looking at the code its referencing i see:

      ssl_options = {
        :ca_file => @config.ssl_ca_file,
        :verify => @config.ssl_verify,
        :verify => @config.ssl_verify_mode,
        :client_cert => @config.ssl_client_cert,
        :client_key => @config.ssl_client_key
      }

Notice :verify is mentioned twice.

History

#2 Updated by dkliban@redhat.com about 1 year ago

  • Status changed from NEW to MODIFIED

The PR has been merged upstream. New ruby gems have been published.

#3 Updated by dkliban@redhat.com about 1 year ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 57

#4 Updated by bmbouter 11 months ago

  • Sprint/Milestone set to 3.0.0

#5 Updated by bmbouter 11 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

#6 Updated by ggainey 6 months ago

  • Tags Katello added
  • Tags deleted (Katello-P2)

Please register to edit this issue

Also available in: Atom PDF