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

Listing profiles on the Supermarket should return all profiles #1219

Closed
robbkidd opened this issue Oct 13, 2016 · 2 comments
Closed

Listing profiles on the Supermarket should return all profiles #1219

robbkidd opened this issue Oct 13, 2016 · 2 comments
Labels
Type: Bug Feature not working as expected
Milestone

Comments

@robbkidd
Copy link
Contributor

Description

inspec supermarket profiles currently uses the only query parameter the Supermarket tools API offers at the moment: q=. Sadly, this means that the returned list of profiles is limited to the number of tools that mention the words "compliance profile" in their descriptions. So users are not seeing a complete list of profiles available.

InSpec and Platform Version

> inspec version
1.0.0

Supermarket version 2.8.27.

Replication Case

> inspec supermarket profiles
== Available profiles:

 * nathenharvey/tmp-compliance-profile
 * sliim/chef-client-hardening
 * dev-sec/cis-docker-benchmark
 * dev-sec/nginx-hardening
 * hardening/os-hardening
 * hardening/ssh-hardening

Note, this is not a complete list of compliance profiles currently on the Supermarket (10 at the time of this writing). These profiles appear because they contain the words "compliance profile" in their descriptions.

Possible Solutions

After a fix for chef/supermarket#1454 is released, the request parameters for the search in inspec supermarket can be updated. Pending the Supermarket release, the params will probably look like:

def self.profiles(supermarket_url = SUPERMARKET_URL)
      url = "#{supermarket_url}/api/v1/tools-search"
-       _success, data = get(url, { q: 'compliance_profile' })
+       _success, data = get(url, { type: 'compliance_profile' })
      if !data.nil?
@arlimus arlimus added this to the Prioritized milestone Oct 15, 2016
@arlimus arlimus added the Type: Bug Feature not working as expected label Oct 15, 2016
@alexpop
Copy link
Contributor

alexpop commented Oct 26, 2016

Robb, ok to close this issue based on merged PR #1254 and issue #1255 ?

@alexpop alexpop closed this as completed Oct 28, 2016
@robbkidd
Copy link
Contributor Author

robbkidd commented Nov 1, 2016

Tuning back in after Summit, my opinion is that this issue is not fixed. My understanding of PR #1254 is that the results will be whichever compliance profiles that appear in the latest 100 tools updated in a Supermarket. That might yield happy-enough results today, but I don't think it correctly or fully addresses the issue described here.

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

No branches or pull requests

3 participants