-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(PSP-1220): add new cloud account type for Azure Agentless integration #1443
Changes from 3 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,82 @@ | ||
// | ||
// Author:: Ao Zhang (<[email protected]>) | ||
// Copyright:: Copyright 2023, Lacework Inc. | ||
// License:: Apache License, Version 2.0 | ||
// | ||
// Licensed under the Apache License, Version 2.0 (the "License"); | ||
// you may not use this file except in compliance with the License. | ||
// You may obtain a copy of the License at | ||
// | ||
// http://www.apache.org/licenses/LICENSE-2.0 | ||
// | ||
// Unless required by applicable law or agreed to in writing, software | ||
// distributed under the License is distributed on an "AS IS" BASIS, | ||
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
// See the License for the specific language governing permissions and | ||
// limitations under the License. | ||
// | ||
|
||
package api | ||
|
||
// GetAzureSidekick gets a single AzureSidekick integration matching the provided integration guid | ||
func (svc *CloudAccountsService) GetAzureSidekick(guid string) ( | ||
response AzureSidekickIntegrationResponse, | ||
err error, | ||
) { | ||
err = svc.get(guid, &response) | ||
return | ||
} | ||
|
||
// CreateAzureSidekick creates an AzureSidekick Cloud Account integration | ||
func (svc *CloudAccountsService) CreateAzureSidekick(data CloudAccount) ( | ||
response AzureSidekickIntegrationResponse, | ||
err error, | ||
) { | ||
err = svc.create(data, &response) | ||
return | ||
} | ||
|
||
// UpdateAzureSidekick updates a single AzureSidekick integration on the Lacework Server | ||
func (svc *CloudAccountsService) UpdateAzureSidekick(data CloudAccount) ( | ||
response AzureSidekickIntegrationResponse, | ||
err error, | ||
) { | ||
err = svc.update(data.ID(), data, &response) | ||
return | ||
} | ||
|
||
type AzureSidekickIntegrationResponse struct { | ||
Data V2AzureSidekickIntegration `json:"data"` | ||
} | ||
|
||
type AzureSidekickToken struct { | ||
ServerToken string `json:"serverToken"` | ||
Uri string `json:"uri"` | ||
} | ||
|
||
type V2AzureSidekickIntegration struct { | ||
v2CommonIntegrationData | ||
AzureSidekickToken `json:"serverToken"` | ||
Data AzureSidekickData `json:"data"` | ||
} | ||
|
||
type AzureSidekickData struct { | ||
Credentials AzureSidekickCredentials `json:"credentials"` | ||
IntegrationType string `json:"integrationType"` // SUBSCRIPTION or TENANT | ||
SubscriptionId string `json:"subscriptionId"` | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This should be named scanningSubscriptionId, isn't it? |
||
TenantId string `json:"tenantId"` | ||
BlobContainerName string `json:"blobContainerName"` | ||
SubscriptionList string `json:"subscriptionList,omitempty"` | ||
QueryText string `json:"queryText,omitempty"` | ||
ScanFrequency int `json:"scanFrequency"` // in hours | ||
ScanContainers bool `json:"scanContainers"` | ||
ScanHostVulnerabilities bool `json:"scanHostVulnerabilities"` | ||
ScanMultiVolume bool `json:"scanMultiVolume"` | ||
ScanStoppedInstances bool `json:"scanStoppedInstances"` | ||
} | ||
|
||
type AzureSidekickCredentials struct { | ||
ClientID string `json:"clientId"` | ||
ClientSecret string `json:"clientSecret,omitempty"` | ||
CredentialType string `json:"credentialType"` // SharedCredentials or SharedAccess | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Currently AZURE_SIDEKICK integration schema doesn't have credentialType, so adding it would violate the schema. Is this field necessary? If yes, we would have to add it to the schema. |
||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,258 @@ | ||
// | ||
// Author:: Ao Zhang (<[email protected]>) | ||
// Copyright:: Copyright 2023, Lacework Inc. | ||
// License:: Apache License, Version 2.0 | ||
// | ||
// Licensed under the Apache License, Version 2.0 (the "License"); | ||
// you may not use this file except in compliance with the License. | ||
// You may obtain a copy of the License at | ||
// | ||
// http://www.apache.org/licenses/LICENSE-2.0 | ||
// | ||
// Unless required by applicable law or agreed to in writing, software | ||
// distributed under the License is distributed on an "AS IS" BASIS, | ||
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
// See the License for the specific language governing permissions and | ||
// limitations under the License. | ||
// | ||
|
||
package api_test | ||
|
||
import ( | ||
"fmt" | ||
"net/http" | ||
"testing" | ||
|
||
"github.com/lacework/go-sdk/api" | ||
"github.com/lacework/go-sdk/internal/intgguid" | ||
"github.com/lacework/go-sdk/internal/lacework" | ||
"github.com/stretchr/testify/assert" | ||
) | ||
|
||
// These two objects are used to test Create, Get and Update operations. | ||
var ( | ||
azureSidekickData = api.AzureSidekickData{ | ||
IntegrationType: "SUBSCRIPTION", | ||
SubscriptionId: "54321", | ||
TenantId: "98765", | ||
BlobContainerName: "blobContainer", | ||
ScanFrequency: 24, | ||
ScanContainers: true, | ||
ScanHostVulnerabilities: true, | ||
Credentials: api.AzureSidekickCredentials{ | ||
ClientID: "Client123", | ||
CredentialType: "SharedCredentials", | ||
ClientSecret: "Secret", | ||
}, | ||
SubscriptionList: "sub1,sub2", | ||
QueryText: "queryText", | ||
} | ||
|
||
azureUpdatedSidekickData = api.AzureSidekickData{ | ||
IntegrationType: "SUBSCRIPTION", | ||
SubscriptionId: "updated-54321", | ||
TenantId: "updated-98765", | ||
BlobContainerName: "updated-blobContainer", | ||
ScanFrequency: 12, | ||
ScanContainers: false, | ||
ScanHostVulnerabilities: true, | ||
Credentials: api.AzureSidekickCredentials{ | ||
ClientID: "updated-Client123", | ||
CredentialType: "SharedAccess", | ||
ClientSecret: "updated-Secret", | ||
}, | ||
SubscriptionList: "updated-sub1,sub2", | ||
QueryText: "updated-queryText", | ||
} | ||
) | ||
|
||
func TestCloudAccountsAzureSidekickCreate(t *testing.T) { | ||
integration := api.NewCloudAccount("integration_name", api.AzureSidekickCloudAccount, azureSidekickData) | ||
assert.Equal(t, api.AzureSidekickCloudAccount.String(), integration.Type) | ||
|
||
// casting the data interface{} to type AzureSidekickData | ||
integrationData := integration.Data.(api.AzureSidekickData) | ||
|
||
assert.Equal(t, integrationData.IntegrationType, "SUBSCRIPTION") | ||
assert.Equal(t, integrationData.SubscriptionId, "54321") | ||
assert.Equal(t, integrationData.TenantId, "98765") | ||
assert.Equal(t, integrationData.BlobContainerName, "blobContainer") | ||
assert.Equal(t, integrationData.ScanFrequency, 24) | ||
assert.Equal(t, integrationData.ScanContainers, true) | ||
assert.Equal(t, integrationData.ScanHostVulnerabilities, true) | ||
|
||
assert.Equal(t, integrationData.Credentials.ClientID, "Client123") | ||
assert.Equal(t, integrationData.Credentials.ClientSecret, "Secret") | ||
assert.Equal(t, integrationData.Credentials.CredentialType, "SharedCredentials") | ||
} | ||
|
||
func TestCloudAccountsAzureSidekickGet(t *testing.T) { | ||
var ( | ||
intgGUID = intgguid.New() | ||
apiPath = fmt.Sprintf("CloudAccounts/%s", intgGUID) | ||
fakeServer = lacework.MockServer() | ||
) | ||
fakeServer.MockToken("TOKEN") | ||
defer fakeServer.Close() | ||
|
||
fakeServer.MockAPI(apiPath, func(w http.ResponseWriter, r *http.Request) { | ||
assert.Equal(t, "GET", r.Method, "GetAzureSidekick() should be a GET method") | ||
fmt.Fprintf(w, generateCloudAccountResponse(getAzureData(intgGUID, azureSidekickData))) | ||
}) | ||
|
||
c, err := api.NewClient("test", | ||
api.WithToken("TOKEN"), | ||
api.WithURL(fakeServer.URL()), | ||
) | ||
assert.Nil(t, err) | ||
|
||
response, err := c.V2.CloudAccounts.GetAzureSidekick(intgGUID) | ||
assert.Nil(t, err) | ||
assert.NotNil(t, response) | ||
|
||
integration := response.Data | ||
assert.Equal(t, intgGUID, integration.IntgGuid) | ||
assert.Equal(t, "integration_test", integration.Name) | ||
assert.True(t, integration.State.Ok) | ||
|
||
integrationData := integration.Data | ||
assert.Equal(t, "SUBSCRIPTION", integrationData.IntegrationType) | ||
assert.Equal(t, "54321", integrationData.SubscriptionId) | ||
assert.Equal(t, "98765", integrationData.TenantId) | ||
assert.Equal(t, "blobContainer", integrationData.BlobContainerName) | ||
assert.Equal(t, 24, integrationData.ScanFrequency) | ||
assert.Equal(t, true, integrationData.ScanContainers) | ||
assert.Equal(t, true, integrationData.ScanHostVulnerabilities) | ||
assert.Equal(t, "Client123", integrationData.Credentials.ClientID) | ||
assert.Equal(t, "Secret", integrationData.Credentials.ClientSecret) | ||
assert.Equal(t, "SharedCredentials", integrationData.Credentials.CredentialType) | ||
assert.Equal(t, "sub1,sub2", integrationData.SubscriptionList) | ||
assert.Equal(t, "queryText", integrationData.QueryText) | ||
assert.Equal(t, "token_"+integration.IntgGuid, integration.ServerToken) | ||
} | ||
|
||
func TestCloudAccountsAzureSidekickUpdate(t *testing.T) { | ||
var ( | ||
intgGUID = intgguid.New() | ||
apiPath = fmt.Sprintf("CloudAccounts/%s", intgGUID) | ||
fakeServer = lacework.MockServer() | ||
) | ||
fakeServer.MockToken("TOKEN") | ||
defer fakeServer.Close() | ||
|
||
// Step 1 - Start Fake Server to return updated data | ||
fakeServer.MockAPI(apiPath, func(w http.ResponseWriter, r *http.Request) { | ||
assert.Equal(t, "PATCH", r.Method, "UpdateAzureSidekick() should be a PATCH method") | ||
|
||
if assert.NotNil(t, r.Body) { | ||
body := httpBodySniffer(r) | ||
assert.Contains(t, body, intgGUID, "INTG_GUID missing") | ||
assert.Contains(t, body, "integration_test", "cloud account name is missing") | ||
assert.Contains(t, body, "AzureSidekick", "wrong cloud account type") | ||
assert.Contains(t, body, azureSidekickData.Credentials.ClientID, "wrong client ID") | ||
assert.Contains(t, body, azureSidekickData.BlobContainerName, "wrong blob container name") | ||
assert.Contains(t, body, "enabled\":1", "cloud account is not enabled") | ||
} | ||
|
||
fmt.Fprintf(w, generateCloudAccountResponse(getAzureData(intgGUID, azureUpdatedSidekickData))) | ||
}) | ||
|
||
c, err := api.NewClient("test", | ||
api.WithToken("TOKEN"), | ||
api.WithURL(fakeServer.URL()), | ||
) | ||
assert.Nil(t, err) | ||
|
||
// Step 2 - Create New Account | ||
cloudAccount := api.NewCloudAccount("integration_test", | ||
api.AzureSidekickCloudAccount, | ||
azureSidekickData, | ||
) | ||
|
||
integrationData := cloudAccount.Data.(api.AzureSidekickData) | ||
assert.Equal(t, "integration_test", cloudAccount.Name) | ||
assert.Equal(t, "AzureSidekick", cloudAccount.Type) | ||
assert.Equal(t, 1, cloudAccount.Enabled) | ||
assert.Equal(t, "SUBSCRIPTION", integrationData.IntegrationType) | ||
assert.Equal(t, "54321", integrationData.SubscriptionId) | ||
assert.Equal(t, "98765", integrationData.TenantId) | ||
assert.Equal(t, "blobContainer", integrationData.BlobContainerName) | ||
assert.Equal(t, 24, integrationData.ScanFrequency) | ||
assert.Equal(t, true, integrationData.ScanContainers) | ||
assert.Equal(t, true, integrationData.ScanHostVulnerabilities) | ||
assert.Equal(t, "Client123", integrationData.Credentials.ClientID) | ||
assert.Equal(t, "Secret", integrationData.Credentials.ClientSecret) | ||
assert.Equal(t, "SharedCredentials", integrationData.Credentials.CredentialType) | ||
assert.Equal(t, "sub1,sub2", integrationData.SubscriptionList) | ||
assert.Equal(t, "queryText", integrationData.QueryText) | ||
|
||
// Step 3 - Get Updated data from Fake server | ||
cloudAccount.IntgGuid = intgGUID | ||
response, err := c.V2.CloudAccounts.UpdateAzureSidekick(cloudAccount) | ||
assert.Nil(t, err, "Cannot update integration") | ||
assert.NotNil(t, response) | ||
integration := response.Data | ||
assert.Equal(t, intgGUID, integration.IntgGuid) | ||
|
||
integrationData = integration.Data | ||
assert.Equal(t, "integration_test", cloudAccount.Name) | ||
assert.Equal(t, "AzureSidekick", cloudAccount.Type) | ||
assert.Equal(t, 1, cloudAccount.Enabled) | ||
assert.Equal(t, "SUBSCRIPTION", integrationData.IntegrationType) | ||
assert.Equal(t, "updated-54321", integrationData.SubscriptionId) | ||
assert.Equal(t, "updated-98765", integrationData.TenantId) | ||
assert.Equal(t, "updated-blobContainer", integrationData.BlobContainerName) | ||
assert.Equal(t, 12, integrationData.ScanFrequency) | ||
assert.Equal(t, false, integrationData.ScanContainers) | ||
assert.Equal(t, true, integrationData.ScanHostVulnerabilities) | ||
assert.Equal(t, "updated-Client123", integrationData.Credentials.ClientID) | ||
assert.Equal(t, "updated-Secret", integrationData.Credentials.ClientSecret) | ||
assert.Equal(t, "SharedAccess", integrationData.Credentials.CredentialType) | ||
assert.Equal(t, "updated-sub1,sub2", integrationData.SubscriptionList) | ||
assert.Equal(t, "updated-queryText", integrationData.QueryText) | ||
} | ||
|
||
// getAzureData converts integration data to json string | ||
func getAzureData(id string, data api.AzureSidekickData) string { | ||
|
||
scanFrequency := fmt.Sprintf("%d", data.ScanFrequency) | ||
scanContainers := fmt.Sprintf("%t", data.ScanContainers) | ||
scanHostVulnerabilities := fmt.Sprintf("%t", data.ScanHostVulnerabilities) | ||
|
||
return ` | ||
{ | ||
"createdOrUpdatedBy": "[email protected]", | ||
"createdOrUpdatedTime": "2021-06-01T19:28:00.092Z", | ||
"enabled": 1, | ||
"intgGuid": "` + id + `", | ||
"isOrg": 0, | ||
"name": "integration_test", | ||
"state": { | ||
"details": {}, | ||
"lastSuccessfulTime": 1624456896915, | ||
"lastUpdatedTime": 1624456896915, | ||
"ok": true | ||
}, | ||
"type": "AzureSidekick", | ||
"data": { | ||
"credentials": { | ||
"clientId": "` + data.Credentials.ClientID + `", | ||
"clientSecret": "` + data.Credentials.ClientSecret + `", | ||
"credentialType": "` + data.Credentials.CredentialType + `" | ||
}, | ||
"integrationType": "` + data.IntegrationType + `", | ||
"subscriptionId": "` + data.SubscriptionId + `", | ||
"tenantId": "` + data.TenantId + `", | ||
"blobContainerName": "` + data.BlobContainerName + `", | ||
"subscriptionList": "` + data.SubscriptionList + `", | ||
"queryText": "` + data.QueryText + `", | ||
"scanFrequency": ` + scanFrequency + `, | ||
"scanContainers": ` + scanContainers + `, | ||
"scanHostVulnerabilities": ` + scanHostVulnerabilities + ` | ||
}, | ||
"serverToken": { | ||
"serverToken": "token_` + id + `" | ||
} | ||
} | ||
` | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be called integrationLevel?
https://github.com/lacework/rainbow/blob/7b6b998bb4b8ec14f11fb0700e1cff12fce2bf7f/iris/src/main/resources/schema/INTEGRATION_SCHEMA.json#L303C20-L303C20