Error encountered when creating an Enforcer based on the ABAC model: extraneous or missing "in quoted field"

[Harris-H]

I found that the first time I created an Enforcer when the database was empty and checked if the request was fulfilled without any issues, but when the database included these policies, creating an Enforcer would result in an error message: extraneous or missing "in quoted field". What is the problem? Please help me.

I am using an adapter based on MongoDB：

	a, err := mongodbadapter.NewAdapterWithClientOption(mongoClientOption, databaseName)
	if err != nil {
		panic(err)
	}

	e, err := casbin.NewEnforcer("../conf/abac_model.conf", a)

My model:

[request_definition]
r = sub, obj, act

[policy_definition]
p = sub_rule, obj, act

[policy_effect]
e = some(where (p.eft == allow))

[matchers]
m = eval(p.sub_rule) && r.obj == p.obj && r.act == p.act

My policy:

My request(s):

models.UserRole{
		Role: "test_admin",
	}, index, read ---> true

[hsluoyz]

@Harris-H can you provide a full example? Which API did you call? and paste the IDE error directly

[Harris-H]

@Harris-H can you provide a full example? Which API did you call? and paste the IDE error directly

When I call the NewEnforcer() function API, I will report an error: extraneous or missing "in quoted field". However, I will not report an error when I do not have a policy in the database. Testing with request also runs normally, and creating an Enforcer again after adding a policy will report an error.

[slhmy]

Looks like a csv parsing error, I'll take a look.

[hsluoyz]

Hi @slhmy thanks for your contribution! Can you join the QQ group: 546057381 ?

[slhmy]

546057381

Comming

[slhmy]

It's the problem in:

mongodb-adapter/adapter.go

Lines 240 to 259 in 808774d

	func loadPolicyLine(line CasbinRule, model model.Model) error {
	var p = []string{line.PType,
	line.V0, line.V1, line.V2, line.V3, line.V4, line.V5}
	var lineText string
	if line.V5 != "" {
	lineText = strings.Join(p, ", ")
	} else if line.V4 != "" {
	lineText = strings.Join(p[:6], ", ")
	} else if line.V3 != "" {
	lineText = strings.Join(p[:5], ", ")
	} else if line.V2 != "" {
	lineText = strings.Join(p[:4], ", ")
	} else if line.V1 != "" {
	lineText = strings.Join(p[:3], ", ")
	} else if line.V0 != "" {
	lineText = strings.Join(p[:2], ", ")
	}
	return persist.LoadPolicyLine(lineText, model)
	}

When you join the following fields in to a line:

{
  "_id": {
    "$oid": "68318a3be06deabfc910089b"
  },
  "ptype": "p",
  "v0": "alice",
  "v1": "fn(data1, \"data\")",
  "v2": "read",
  "v3": "",
  "v4": "",
  "v5": ""
}

It will got an invalid csv line like: p,alice,fn(data1, "data"),read, which should be p, alice,"fn(data1, ""data"")", read if we want parse the line in csv form.

In the fix, I will use the same writing as it in gorm adapter https://github.com/casbin/gorm-adapter/blob/master/adapter.go#L466-L482 , which is skipping the csv parsing step in db adapter by using persist.LoadPolicyArray instead of persist.LoadPolicyLine.