#18 merge request comments

cmd/policy/main.go

@@ -74,10 +74,10 @@ func main() {
 	// register rego extension functions
 	{
 		cacheFuncs := regofunc.NewCacheFuncs(cfg.Cache.Addr, httpClient())
-		DIDResolverFuncs := regofunc.NewDIDResolverFuncs(cfg.DIDResolver.Addr, httpClient())
+		didResolverFuncs := regofunc.NewDIDResolverFuncs(cfg.DIDResolver.Addr, httpClient())
 		regofunc.Register("cacheGet", rego.Function3(cacheFuncs.CacheGetFunc()))
 		regofunc.Register("cacheSet", rego.Function4(cacheFuncs.CacheSetFunc()))
-		regofunc.Register("DIDResolve", rego.Function1(DIDResolverFuncs.Resolve()))
+		regofunc.Register("didResolve", rego.Function1(didResolverFuncs.Resolve()))
 		regofunc.Register("strictBuiltinErrors", rego.StrictBuiltinErrors(true))
 	}
 

internal/config/config.go

@@ -6,7 +6,7 @@ type Config struct {
 	HTTP        httpConfig
 	Mongo       mongoConfig
 	Cache       cacheConfig
-	DIDResolver DIDResolverConfig
+	DIDResolver didResolverConfig
 
 	LogLevel string `envconfig:"LOG_LEVEL" default:"INFO"`
 }
@@ -23,7 +23,7 @@ type cacheConfig struct {
 	Addr string `envconfig:"CACHE_ADDR" required:"true"`
 }
 
-type DIDResolverConfig struct {
+type didResolverConfig struct {
 	Addr string `envconfig:"DID_RESOLVER_ADDR" required:"true"`
 }
 

internal/regofunc/did_resolver.go

@@ -36,7 +36,7 @@ func (dr *DIDResolverFuncs) Resolve() (*rego.Function, rego.Builtin1) {
 				return nil, fmt.Errorf("invalid DID: %s", err)
 			}
 			if DID == "" {
-				return nil, errors.New("DID can not be empty")
+				return nil, errors.New("DID cannot be empty")
 			}
 
 			req, err := http.NewRequest("GET", dr.resolverAddr+"/1.0/identifiers/"+DID, nil)