fix: avoid direct access to proto field perms.Account and perms.Permissions (#4552)

Pantani · web-flow · commit c7a4bc285338 · 2025-03-03T09:16:57.000+01:00
* avoid direct access to proto field `perms.Account` and `perms.Permissions`

* add changelog
diff --git a/changelog.md b/changelog.md
@@ -58,6 +58,7 @@
 - [#4091](https://github.com/ignite/cli/pull/4091) Fix race conditions in the plugin logic
 - [#4128](https://github.com/ignite/cli/pull/4128) Check for duplicate proto fields in config
 - [#4402](https://github.com/ignite/cli/pull/4402) Fix gentx parser into the cosmosutil package
+- [#4552](https://github.com/ignite/cli/pull/4552) Avoid direct access to proto field `perms.Account` and `perms.Permissions`
 
 ## [`v28.8.1`](https://github.com/ignite/cli/releases/tag/v28.8.1)
 
diff --git a/ignite/templates/app/files-minimal/app/app.go.plush b/ignite/templates/app/files-minimal/app/app.go.plush
@@ -218,7 +218,7 @@ func (app *App) RegisterAPIRoutes(apiSvr *api.Server, apiConfig config.APIConfig
 func GetMaccPerms() map[string][]string {
 	dup := make(map[string][]string)
 	for _, perms := range moduleAccPerms {
-		dup[perms.Account] = perms.Permissions
+		dup[perms.GetAccount()] = perms.GetPermissions()
 	}
 
 	return dup
diff --git a/ignite/templates/app/files/app/app.go.plush b/ignite/templates/app/files/app/app.go.plush
@@ -275,7 +275,7 @@ func (app *App) RegisterAPIRoutes(apiSvr *api.Server, apiConfig config.APIConfig
 func GetMaccPerms() map[string][]string {
 	dup := make(map[string][]string)
 	for _, perms := range moduleAccPerms {
-		dup[perms.Account] = perms.Permissions
+		dup[perms.GetAccount()] = perms.GetPermissions()
 	}
 
 	return dup

Original file line number	Diff line number	Diff line change
`@@ -218,7 +218,7 @@ func (app App) RegisterAPIRoutes(apiSvr api.Server, apiConfig config.APIConfig`
`218`	`218`	`func GetMaccPerms() map[string][]string {`
`219`	`219`	`dup := make(map[string][]string)`
`220`	`220`	`for _, perms := range moduleAccPerms {`
`221`		`- dup[perms.Account] = perms.Permissions`
	`221`	`+ dup[perms.GetAccount()] = perms.GetPermissions()`
`222`	`222`	`}`
`223`	`223`
`224`	`224`	`return dup`
Original file line number	Diff line number	Diff line change
`@@ -275,7 +275,7 @@ func (app App) RegisterAPIRoutes(apiSvr api.Server, apiConfig config.APIConfig`
`275`	`275`	`func GetMaccPerms() map[string][]string {`
`276`	`276`	`dup := make(map[string][]string)`
`277`	`277`	`for _, perms := range moduleAccPerms {`
`278`		`- dup[perms.Account] = perms.Permissions`
	`278`	`+ dup[perms.GetAccount()] = perms.GetPermissions()`
`279`	`279`	`}`
`280`	`280`
`281`	`281`	`return dup`