Report for github.com/GoodwayGroup/gw-aws-audit
A+ Excellent! Found 16 issues across 38 files
gofmt100%
Gofmt formats Go programs. We run gofmt -s on your code, where -s is for the "simplify" command
No problems detected. Good job!
go_vet100%
go vet examines Go source code and reports suspicious constructs, such as Printf calls whose arguments do not align with the format string.
No problems detected. Good job!
gocyclo86%
Gocyclo calculates cyclomatic complexities of functions in Go source code. The cyclomatic complexity of a function is calculated according to the following rules: 1 is the base complexity of a function +1 for each 'if', 'for', 'case', '&&' or '||' Go Report Card warns on functions with cyclomatic complexity > 15.
-
-
gw-aws-audit/sg/get.go
- Line 127: warning: cyclomatic complexity 16 of function detectAttachedSecurityGroups() is high (> 15) (gocyclo)
-
-
gw-aws-audit/iam/action.go
- Line 32: warning: cyclomatic complexity 26 of function keyActions() is high (> 15) (gocyclo)
-
-
gw-aws-audit/main.go
- Line 28: warning: cyclomatic complexity 24 of function main() is high (> 15) (gocyclo)
-
-
gw-aws-audit/sg/aws.go
- Line 110: warning: cyclomatic complexity 20 of function GenerateMappedEC2Report() is high (> 15) (gocyclo)
-
-
gw-aws-audit/sg/helper.go
- Line 98: warning: cyclomatic complexity 17 of function processSecurityGroups() is high (> 15) (gocyclo)
golint68%
Golint is a linter for Go source code.
-
-
gw-aws-audit/ec2/helper.go
- Line 41: warning: exported type Info should have comment or be unexported (golint)
- Line 49: warning: exported function GetEC2IPs should have comment or be unexported (golint)
- Line 80: warning: exported function GetInterfaceIPs should have comment or be unexported (golint)
-
-
gw-aws-audit/s3/clear_bucket.go
- Line 17: warning: comment on exported function ClearBucketObjects should be of the form "ClearBucketObjects ..." (golint)
-
-
gw-aws-audit/s3/cost_tag.go
- Line 16: warning: comment on exported function AddCostTag should be of the form "AddCostTag ..." (golint)
-
-
gw-aws-audit/s3/metrics.go
- Line 18: warning: comment on exported function GetBucketMetrics should be of the form "GetBucketMetrics ..." (golint)
-
-
gw-aws-audit/iam/cmd.go
- Line 14: warning: exported var ActionUserKeys should have comment or be unexported (golint)
-
-
gw-aws-audit/iam/user.go
- Line 14: warning: exported type Permission should have comment or be unexported (golint)
- Line 20: warning: exported type User should have comment or be unexported (golint)
- Line 38: warning: exported method User.ARN should have comment or be unexported (golint)
- Line 42: warning: exported method User.UserName should have comment or be unexported (golint)
- Line 46: warning: exported method User.ID should have comment or be unexported (golint)
- Line 83: warning: exported method User.HasConsoleAccess should have comment or be unexported (golint)
- Line 87: warning: exported method User.DisableConsoleAccess should have comment or be unexported (golint)
- Line 105: warning: exported method User.LastLogin should have comment or be unexported (golint)
- Line 109: warning: exported method User.LastLoginDuration should have comment or be unexported (golint)
- Line 116: warning: exported method User.CreatedDate should have comment or be unexported (golint)
- Line 120: warning: exported method User.CreatedDateDuration should have comment or be unexported (golint)
- Line 127: warning: exported method User.GetAccessKeys should have comment or be unexported (golint)
- Line 169: warning: exported method User.AccessKeys should have comment or be unexported (golint)
- Line 173: warning: exported method User.HasAccessKeys should have comment or be unexported (golint)
- Line 177: warning: exported method User.AccessKeysCount should have comment or be unexported (golint)
- Line 181: warning: exported method User.GetPermissions should have comment or be unexported (golint)
- Line 277: warning: exported method User.HasPermissions should have comment or be unexported (golint)
- Line 281: warning: exported method User.Permissions should have comment or be unexported (golint)
- Line 285: warning: exported method User.PermissionsCount should have comment or be unexported (golint)
- Line 289: warning: exported method User.Groups should have comment or be unexported (golint)
- Line 293: warning: exported method User.Policies should have comment or be unexported (golint)
- Line 297: warning: exported method User.InlinePolicies should have comment or be unexported (golint)
- Line 301: warning: comment on exported method User.CheckStatus should be of the form "CheckStatus ..." (golint)
- Line 321: warning: exported method User.FormattedCheckStatus should have comment or be unexported (golint)
- Line 334: warning: exported method User.FormattedLastLoginDateDuration should have comment or be unexported (golint)
-
-
gw-aws-audit/lib/response.go
- Line 12: warning: comment on exported function HandleResponse should be of the form "HandleResponse ..." (golint)
-
-
gw-aws-audit/lib/session.go
- Line 19: warning: exported type Session should have comment or be unexported (golint)
- Line 44: warning: exported method Session.Session should have comment or be unexported (golint)
- Line 48: warning: exported method Session.GetIAMClient should have comment or be unexported (golint)
- Line 52: warning: exported method Session.GetEC2Client should have comment or be unexported (golint)
- Line 56: warning: exported method Session.GetRDSClient should have comment or be unexported (golint)
- Line 60: warning: exported method Session.GetS3Client should have comment or be unexported (golint)
- Line 64: warning: exported method Session.GetCloudWatchClient should have comment or be unexported (golint)
-
-
gw-aws-audit/iam/access_key.go
- Line 10: warning: exported type AccessKey should have comment or be unexported (golint)
- Line 18: warning: exported method AccessKey.ID should have comment or be unexported (golint)
- Line 22: warning: exported method AccessKey.UserName should have comment or be unexported (golint)
- Line 26: warning: exported method AccessKey.Status should have comment or be unexported (golint)
- Line 30: warning: exported method AccessKey.CreatedDate should have comment or be unexported (golint)
- Line 34: warning: exported method AccessKey.LastUsedDate should have comment or be unexported (golint)
- Line 38: warning: exported method AccessKey.LastUsedService should have comment or be unexported (golint)
- Line 42: warning: exported method AccessKey.Deactivate should have comment or be unexported (golint)
- Line 61: warning: exported method AccessKey.Activate should have comment or be unexported (golint)
- Line 80: warning: exported method AccessKey.Delete should have comment or be unexported (golint)
-
-
gw-aws-audit/sg/aws.go
- Line 14: warning: comment on exported function GenerateExternalAWSIPReport should be of the form "GenerateExternalAWSIPReport ..." (golint)
- Line 110: warning: exported function GenerateMappedEC2Report should have comment or be unexported (golint)
-
-
gw-aws-audit/lib/metrics.go
- Line 3: warning: comment on exported type Metrics should be of the form "Metrics ..." (with optional leading article) (golint)
-
-
gw-aws-audit/rds/monitoring.go
- Line 12: warning: comment on exported function ListMonitoringEnabled should be of the form "ListMonitoringEnabled ..." (golint)
ineffassign100%
IneffAssign detects ineffectual assignments in Go code.
No problems detected. Good job!