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

map url to https for compliance plugin #1471

Merged
merged 1 commit into from
Feb 8, 2017
Merged

map url to https for compliance plugin #1471

merged 1 commit into from
Feb 8, 2017

Conversation

arlimus
Copy link
Contributor

@arlimus arlimus commented Feb 8, 2017

As submitted by @billmeyer , Thank you!

This solves an issue where entering the hostname without the schema would result in the following error:

/pub/git/inspec/lib/bundles/inspec-compliance/http.rb:70:in `send_request': Unable to parse URI: hostname/compliance/profiles/username (RuntimeError)
	from /pub/git/inspec/lib/bundles/inspec-compliance/http.rb:20:in `get'
	from /pub/git/inspec/lib/bundles/inspec-compliance/api.rb:16:in `profiles'
	from /pub/git/inspec/lib/bundles/inspec-compliance/cli.rb:94:in `profiles'
	from /usr/lib/ruby/gems/2.3.0/gems/thor-0.19.4/lib/thor/command.rb:27:in `run'
	from /usr/lib/ruby/gems/2.3.0/gems/thor-0.19.4/lib/thor/invocation.rb:126:in `invoke_command'
	from /usr/lib/ruby/gems/2.3.0/gems/thor-0.19.4/lib/thor.rb:369:in `dispatch'
	from /usr/lib/ruby/gems/2.3.0/gems/thor-0.19.4/lib/thor/invocation.rb:115:in `invoke'
	from /usr/lib/ruby/gems/2.3.0/gems/thor-0.19.4/lib/thor.rb:242:in `block in subcommand'
	from /usr/lib/ruby/gems/2.3.0/gems/thor-0.19.4/lib/thor/command.rb:27:in `run'
	from /usr/lib/ruby/gems/2.3.0/gems/thor-0.19.4/lib/thor/invocation.rb:126:in `invoke_command'
	from /usr/lib/ruby/gems/2.3.0/gems/thor-0.19.4/lib/thor.rb:369:in `dispatch'
	from /usr/lib/ruby/gems/2.3.0/gems/thor-0.19.4/lib/thor/base.rb:444:in `start'
	from bin/inspec:12:in `<main>'

Map the schema to https by default.

@arlimus
Copy link
Contributor Author

arlimus commented Feb 8, 2017

Issue submitted here #1473

Signed-off-by: Dominik Richter <dominik.richter@gmail.com>
@arlimus arlimus merged commit 607601e into master Feb 8, 2017
@arlimus arlimus deleted the dr/scheme-url branch February 8, 2017 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Feature not working as expected
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant