Jira (FACT-3207) Don't rescue NoMethodError

4 views
Skip to first unread message

Josh Cooper (Jira)

unread,
Jun 6, 2023, 11:11:02 AM6/6/23
to puppe...@googlegroups.com
Josh Cooper created an issue
 
Facter / Bug FACT-3207
Don't rescue NoMethodError
Issue Type: Bug Bug
Assignee: Unassigned
Created: 2023/06/06 8:10 AM
Priority: Normal Normal
Reporter: Josh Cooper

The base resolver rescues NoMethodError, which hides bugs in resolvers. For example, if I disable the rescue, exposing a bug in the dmi resolver:

$ bundle exec facter --debug
[2023-06-06 08:07:21.541262 ] DEBUG Facter::Resolvers::Linux::DmiBios - Resolving fact product_serial, but got undefined method `encode' for nil:NilClass
 
                                                   .encode('UTF-8', invalid: :replace)
                                                   ^^^^^^^ at /home/josh/work/facter/lib/facter/resolvers/dmi.rb:39:in `read_facts' 

Add Comment Add Comment
 
This message was sent by Atlassian Jira (v8.20.21#820021-sha1:38274c8)
Atlassian logo

Josh Cooper (Jira)

unread,
Jun 6, 2023, 11:17:03 AM6/6/23
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
The base resolver rescues NoMethodError, which hides bugs in resolvers. For example, if I disable the rescue, exposing it exposes a bug in the dmi resolver:

{noformat}

$ bundle exec facter --debug
[2023-06-06 08:07:21.541262 ] DEBUG Facter::Resolvers::Linux::DmiBios - Resolving fact product_serial, but got undefined method `encode' for nil:NilClass

                     .encode('UTF-8', invalid: :replace)
                     ^^^^^^^ at /home/josh/work/facter/lib/facter/resolvers/dmi.rb:39:in `read_facts'
{noformat}

Michael Hashizume (Jira)

unread,
Jun 6, 2023, 4:09:02 PM6/6/23
to puppe...@googlegroups.com
Michael Hashizume updated an issue
Change By: Michael Hashizume
Epic Link: PUP-11660
Reply all
Reply to author
Forward
0 new messages