Skip to content

Commit

Permalink
Merge pull request #121 from ns1/CORE-656/accout-permission
Browse files Browse the repository at this point in the history
CORE-656/Manage_ip_whitelist permission
  • Loading branch information
mburtless authored Mar 4, 2021
2 parents 0049cc0 + a237487 commit 6180238
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 2 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,9 @@
## 2.5.0 (Unreleased)

## 2.4.4
FEATURES:
* Added missing manage_ip_whitelist permission to the account permissions section

## 2.4.3
FEATURES:
* Added support for tagging in DDI on records and zones
Expand Down
2 changes: 1 addition & 1 deletion rest/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import (
)

const (
clientVersion = "2.4.3"
clientVersion = "2.4.4"

defaultEndpoint = "https://api.nsone.net/v1/"
defaultShouldFollowPagination = true
Expand Down
1 change: 1 addition & 0 deletions rest/model/account/permissions.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ type PermissionsAccount struct {
ManageAccountSettings bool `json:"manage_account_settings"`
ViewActivityLog bool `json:"view_activity_log"`
ViewInvoices bool `json:"view_invoices"`
ManageIPWhitelist bool `json:"manage_ip_whitelist"`
}

// PermissionsSecurity wraps a User's "permissions.security" attribute.
Expand Down
4 changes: 3 additions & 1 deletion rest/model/account/user_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,8 @@ func TestUnmarshalUsers(t *testing.T) {
"manage_account_settings": false,
"view_activity_log": false,
"view_invoices": false,
"manage_users": false
"manage_users": false,
"manage_ip_whitelist": true
},
"monitoring": {
"manage_lists": false,
Expand Down Expand Up @@ -135,6 +136,7 @@ func TestUnmarshalUsers(t *testing.T) {
ViewActivityLog: false,
ViewInvoices: false,
ManageUsers: false,
ManageIPWhitelist: true,
},
Monitoring: PermissionsMonitoring{
ManageLists: false,
Expand Down

0 comments on commit 6180238

Please sign in to comment.