Skip to content

Added sensu check and gem files - #1

Merged
bennibu merged 5 commits into
masterfrom
feature/sensu-check
Aug 1, 2017
Merged

Added sensu check and gem files#1
bennibu merged 5 commits into
masterfrom
feature/sensu-check

Conversation

@bennibu

@bennibu bennibu commented Jul 31, 2017

Copy link
Copy Markdown
Contributor

No description provided.

@bennibu
bennibu requested a review from ushi-as July 31, 2017 10:46
Comment thread bin/check-container-vulnerabilities.rb Outdated
long: '--quayio-token TOKEN'

def run
result = Quayio::Scanner::Check.new(config[:docker_url],

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vielleicht lieber:

status, message = Check.run

Comment thread bin/console Outdated
# Pry.start

require 'irb'
IRB.start(__FILE__)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

kann vermutlcih raus.

Comment thread bin/setup Outdated

bundle install

# Do any other automated setup that you need to do here

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vermutlch auch raus.

Comment thread lib/quayio/scanner/image.rb Outdated
def high_vulnerabilities_present?
raw_scan['data']['Layer']['Features'].detect do |f|
f['Vulnerabilities'] &&
f['Vulnerabilities'].detect { |v| v['Severity'] == 'High' }

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Critical

Comment thread lib/quayio/scanner/image.rb Outdated
JSON.parse(
RestClient.get("https://quay.io/api/v1/repository/#{repo}/image",
authorization: "Bearer #{quayio_token}", accept: :json)
)['images'].detect { |i| i['tags'].include?(tag) }

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bennibu
bennibu force-pushed the feature/sensu-check branch from d2acc5e to 98a2b36 Compare July 31, 2017 11:48
@bennibu

bennibu commented Jul 31, 2017

Copy link
Copy Markdown
Contributor Author

@ushi-as Done.

@bennibu
bennibu merged commit daca182 into master Aug 1, 2017
@bennibu
bennibu deleted the feature/sensu-check branch August 1, 2017 07:40
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.

3 participants