beenull/crowdsec
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

apiserver: remove cached field isEnrolled (#2744)

Browse source 
not worth it just to avoid parsing a string twice
This commit is contained in:
mmetc 2024-01-16 09:14:33 +01:00 committed by GitHub
parent 6acbcb0a33
commit c6e4762f28
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
pkg/apiserver/apiserver.go
View file

@ -43,7 +43,6 @@ type APIServer struct {
papi *Papi
httpServerTomb tomb.Tomb
consoleConfig *csconfig.ConsoleConfig
isEnrolled bool
}
func recoverFromPanic(c *gin.Context) {
@ -226,7 +225,6 @@ func NewServer(config *csconfig.LocalApiServerCfg) (*APIServer, error) {
var (
apiClient *apic
papiClient *Papi
isMachineEnrolled = false
)
controller.AlertsAddChan = nil
@ -245,8 +243,6 @@ func NewServer(config *csconfig.LocalApiServerCfg) (*APIServer, error) {
controller.AlertsAddChan = apiClient.AlertsAddChan
if apiClient.apiClient.IsEnrolled() {
isMachineEnrolled = true
log.Infof("Machine is enrolled in the console, Loading PAPI Client")
papiClient, err = NewPAPI(apiClient, dbClient, config.ConsoleConfig, *config.PapiLogLevel)
@ -279,7 +275,6 @@ func NewServer(config *csconfig.LocalApiServerCfg) (*APIServer, error) {
papi: papiClient,
httpServerTomb: tomb.Tomb{},
consoleConfig: config.ConsoleConfig,
isEnrolled: isMachineEnrolled,
}, nil
}
@ -321,7 +316,7 @@ func (s *APIServer) Run(apiReady chan bool) error {
})
//csConfig.API.Server.ConsoleConfig.ShareCustomScenarios
if s.isEnrolled {
if s.apic.apiClient.IsEnrolled() {
if s.consoleConfig.ConsoleManagement != nil && *s.consoleConfig.ConsoleManagement {
if s.papi.URL != "" {
log.Infof("Starting PAPI decision receiver")