inspec/lib/utils/file_reader.rb
eramoto c7e87ca3e3 Unify method in which file content is read across all resources (#2359)
* Create file-check functionality into utility file

There are the similar issues as PR #2302. Almost resources return false
positives when a file does not exist or is not read.

* Replace to file-check functionality
* Fix dh_params and x509_certificate resources

If a file is empty, OpenSSL::PKey::DH and OpenSSL::X509::Certificate have
raised an exception and have skipped the inspection. Thus x509_certificate
and dh_params resources are not allowed to read a empty file.

* to_s of shadow expects filters is not nil
* Remove workaround of sshd_config

Removes the workaround of sshd_config since Travis CI fails due to a bug
of dev-sec/ssh-baseline and the PR #100 will fix it.

* Use init block variable in methods

Signed-off-by: ERAMOTO Masaya <eramoto.masaya@jp.fujitsu.com>
2018-03-22 08:25:45 -04:00

25 lines
752 B
Ruby

# encoding: utf-8
# author: ERAMOTO Masaya
module FileReader
def read_file_content(path, allow_empty = false)
# these are currently ResourceSkipped to maintain consistency with the resource
# pre-refactor (which used skip_resource). These should likely be changed to
# ResourceFailed during a major version bump.
file = inspec.file(path)
if !file.file?
raise Inspec::Exceptions::ResourceSkipped, "Can't find file: #{path}"
end
raw_content = file.content
if raw_content.nil?
raise Inspec::Exceptions::ResourceSkipped, "Can't read file: #{path}"
end
if !allow_empty && raw_content.empty?
raise Inspec::Exceptions::ResourceSkipped, "File is empty: #{path}"
end
raw_content
end
end