Issues • rudderlabs/rudder-server

String concatenation can be simplified

GO-R4003

Anti-pattern

Major

a month ago — a month old

Seen in 1 file

1

Redundant conversion between `string` and `[]byte`

CRT-A0007

Anti-pattern

Major

2 months ago — 2 years old

Seen in 1 file

1

Use `%q` to quote a string in `fmt.Sprintf` format specifiers

GO-R4002

Anti-pattern

Minor

25 days ago — 2 years old

Seen in 3 files

6

Use `http.NoBody` instead of `nil` in `http.NewRequest` calls

GO-R4001

Anti-pattern

Major

a month ago — 2 years old

Seen in 1 file

1

Empty slice literal used to declare a variable

GO-W1027

Anti-pattern

Minor

22 days ago — a year old

Seen in 2 files

2

Potential issue in `filepath.Join()` function calls

GO-W4001

Anti-pattern

Major

15 days ago — 8 months old

Seen in 3 files

13

Simplify `if` statement for single bool judgment

GO-R1004

Anti-pattern

Minor

2 months ago — 2 months old

Seen in 1 file

1

Function literal can be simplified

CRT-A0018

Anti-pattern

Major

2 months ago — 3 years old

Seen in 1 file

1

Redundant else-blocks can be eliminated

RVV-A0009

Anti-pattern

Major

2 months ago — 3 years old

Seen in 1 file

3

Empty string test can be improved

CRT-A0004

Anti-pattern

Major

a month ago — 3 years old

Seen in 4 files

5

Function with cyclomatic complexity higher than threshold

GO-R1005

Anti-pattern

Minor

12 days ago — 8 months old

Seen in 29 files

44

Dot imports are discouraged

SCC-ST1001

Anti-pattern

Major

a month ago — 2 months old

Seen in 5 files

5

Importing the same package multiple times

GO-W5021

Anti-pattern

Minor

15 days ago — a year old

Seen in 2 files

2

`switch` with single case can be rewritten as `if` or `if-else`

CRT-A0014

Anti-pattern

Minor

15 days ago — 3 years old

Seen in 3 files

3

`context.Context` should be the first param

RVV-A0002

Anti-pattern

Major

a month ago — 3 years old

Seen in 3 files

5

rudderlabs / rudder-server

String concatenation can be simplified

Redundant conversion between string and []byte

Use %q to quote a string in fmt.Sprintf format specifiers

Use http.NoBody instead of nil in http.NewRequest calls

Empty slice literal used to declare a variable

Potential issue in filepath.Join() function calls

Simplify if statement for single bool judgment

Function literal can be simplified

Redundant else-blocks can be eliminated

Empty string test can be improved

Function with cyclomatic complexity higher than threshold

Dot imports are discouraged

Importing the same package multiple times

switch with single case can be rewritten as if or if-else

context.Context should be the first param

Redundant conversion between `string` and `[]byte`

Use `%q` to quote a string in `fmt.Sprintf` format specifiers

Use `http.NoBody` instead of `nil` in `http.NewRequest` calls

Potential issue in `filepath.Join()` function calls

Simplify `if` statement for single bool judgment

`switch` with single case can be rewritten as `if` or `if-else`

`context.Context` should be the first param