[Bug] err not returned in Func LoadPolicyArray

[GerryLon]

Casbin version: v2.52.2
Bug: err not returned in Func LoadPolicyArray

[casbin-bot]

@tangyang9464 @JalinWang

[hsluoyz]

@JalinWang plz work on it

/cc @tangyang9464

[GerryLon]

why not allow empty v5 field?
I use v5 field as a condition(value can be any string), empty v5 field means "any conditioin can pass(like any client ip)".
I think empty string as any condition is general.

[JalinWang]

why not allow empty v5 field? I use v5 field as a condition(value can be any string), empty v5 field means "any conditioin can pass(like any client ip)". I think empty string as any condition is general.

@GerryLon Which adapter?

[GerryLon]

why not allow empty v5 field? I use v5 field as a condition(value can be any string), empty v5 field means "any conditioin can pass(like any client ip)". I think empty string as any condition is general.

@GerryLon Which adapter?

github.com/casbin/mongodb-adapter/v3 v3.3.0

	"github.com/casbin/casbin/v2/persist"
	mongodbadapter "github.com/casbin/mongodb-adapter/v3"

[GerryLon]

why not allow empty v5 field? I use v5 field as a condition(value can be any string), empty v5 field means "any conditioin can pass(like any client ip)". I think empty string as any condition is general.

@GerryLon Which adapter?

my main codes like below:

package auth

import (
  "github.com/casbin/casbin/v2"
  mongodbadapter "github.com/casbin/mongodb-adapter/v3"
)

func InitCasbin() {
  // 定义基于mongodb的casbin适配器
  adapter, err = mongodbadapter.NewAdapterWithClientOption(mongoClientOption, databaseName)
  if err != nil {
    log.ErrorContext(ctx, err)
    return nil, err
  }
  casbinModel, err = casbinmodel.NewModelFromString(mconf)
  if err != nil {
    log.ErrorContext(ctx, err)
    return nil, err
  }
  enforcer, err = casbin.NewSyncedEnforcer(casbinModel, adapter)
  if err != nil {
    log.ErrorContext(ctx, err)
    return nil, err
  }

  // load policy from db or file into memory
  if err = enforcer.LoadPolicy(); err != nil {
    log.ErrorContext(ctx, err)
    return nil, err
  }
}

when I debug above codes, in enforcer.LoadPolicy() -> ... -> LoadPolicyArray()` error is not nil:

[hsluoyz]

why not allow empty v5 field? I use v5 field as a condition(value can be any string), empty v5 field means "any conditioin can pass(like any client ip)". I think empty string as any condition is general.

@GerryLon are you reporting a new issue other than this one? If yes, plz create a new issue here: https://github.com/casbin/mongodb-adapter/issues

[JalinWang]

@JalinWang plz work on it

/cc @tangyang9464

@hsluoyz All the adapters depend on this function without error handling. Should we modify them all?

[hsluoyz]

We can add error as return value, but the adapters don't need to be modified to handle the return value