end
def is_valid_subscription?(content)
- content.length != 0 &&
- content.start_with?("---\n") &&
- YAML.load(content)
+ return false if content.length == 0
+ return false unless content.start_with?("---\n")
+ begin
+ data = YAML.load(content)
+ rescue ArgumentError
+ # Parse error
+ return false
+ end
+ ['name', 'email'].each do |key|
+ return false unless data.include?(key)
+ end
+ true
end
def is_valid_subscription_file?(file)