Cadoles/bouncer
Cadoles
/
bouncer
10
0
Fork 0
Code Issues 7 Pull Requests 1 Projects Releases 3 Wiki Activity

feat(authn) : add case to test multiples CIDR #34 #35

Open
mlamalle wants to merge 1 commits from issue-4 into develop
pull from: issue-4
merge into: Cadoles:develop
Conversation 0 Commits 1 Files Changed 1 +17
1 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
internal/proxy/director/layer/authn/network/authenticator_test.go
View File

@ -39,6 +39,23 @@ func TestMatchAuthorizedCIDRs(t *testing.T) {
}, },
ExpectedResult: false, ExpectedResult: false,
}, },
{
RemoteHostPort: "192.168.1.15:43349",
AuthorizedCIDRs: []string{
"192.168.1.5/32",
"192.168.1.0/24",
},
ExpectedResult: true,
wpetit marked this conversation as resolved Outdated
wpetit commented 2024-08-08 08:53:13 +02:00
Outdated
Review
Copy Link

Le résultat attendu me semble erroné. L'IP 192.168.1.15 est bien contenu dans le réseau 192.168.1.0/24, le résultat sera donc true. Est ce une erreur ?

Le résultat attendu me semble erroné. L'IP `192.168.1.15` est bien contenu dans le réseau `192.168.1.0/24`, le résultat sera donc `true`. Est ce une erreur ?
},
{
RemoteHostPort: "192.168.1.15:43349",
AuthorizedCIDRs: []string{
"192.168.1.5/32",
"192.168.1.6/32",
"192.168.1.7/32",
},
ExpectedResult: false,
},
} }
auth := Authenticator{} auth := Authenticator{}