Please file an issue if we’re missing support for a popular package.

I found this in docs.

go-tools/ir/exits.go

Lines 22 to 60 in 82c7dc9

    
           case "github.com/sirupsen/logrus": 
        
           	switch obj.(*types.Func).FullName() { 
        
           	case "(*github.com/sirupsen/logrus.Logger).Exit": 
        
           		// Technically, this method does not unconditionally exit 
        
           		// the process. It dynamically calls a function stored in 
        
           		// the logger. If the function is nil, it defaults to 
        
           		// os.Exit. 
        
           		// 
        
           		// The main intent of this method is to terminate the 
        
           		// process, and that's what the vast majority of people 
        
           		// will use it for. We'll happily accept some false 
        
           		// negatives to avoid a lot of false positives. 
        
           		fn.WillExit = true 
        
           		return 
        
           	case "(*github.com/sirupsen/logrus.Logger).Panic", 
        
           		"(*github.com/sirupsen/logrus.Logger).Panicf", 
        
           		"(*github.com/sirupsen/logrus.Logger).Panicln": 
        
           		// These methods will always panic, but that's not 
        
           		// statically known from the code alone, because they 
        
           		// take a detour through the generic Log methods. 
        
           		fn.WillUnwind = true 
        
           		return 
        
           	case "(*github.com/sirupsen/logrus.Entry).Panicf", 
        
           		"(*github.com/sirupsen/logrus.Entry).Panicln": 
        
           		// Entry.Panic has an explicit panic, but Panicf and 
        
           		// Panicln do not, relying fully on the generic Log 
        
           		// method. 
        
           		fn.WillUnwind = true 
        
           		return 
        
           	case "(*github.com/sirupsen/logrus.Logger).Log", 
        
           		"(*github.com/sirupsen/logrus.Logger).Logf", 
        
           		"(*github.com/sirupsen/logrus.Logger).Logln": 
        
           		// TODO(dh): we cannot handle these case. Whether they 
        
           		// exit or unwind depends on the level, which is set 
        
           		// via the first argument. We don't currently support 
        
           		// call-site-specific exit information. 
        
           	}

And staticcheck only supports logrus. I think zerolog can be added.

@ainar-g Could you give guide how to add terminating functions to configuration?

Please include support for zerolog Panic (nil pointer check) #793

Description

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions