Skip to content

Commit 0c0c90c

Browse files
committed
feat: changed some resources fields
1 parent de76878 commit 0c0c90c

File tree

8 files changed

+31
-13
lines changed

8 files changed

+31
-13
lines changed

cloudql/github/table_github_organization_collaborator.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,14 @@ func gitHubOrganizationCollaborators() []*plugin.Column {
3838

3939
{
4040
Name: "user_login",
41-
Type: proto.ColumnType_JSON,
41+
Type: proto.ColumnType_STRING,
4242
Description: "The login details of the collaborator.",
4343
Transform: transform.FromField("Description.UserLogin")},
44+
{
45+
Name: "user_id",
46+
Type: proto.ColumnType_STRING,
47+
Description: "The id of the collaborator.",
48+
Transform: transform.FromField("Description.UserID")},
4449
}
4550

4651
return tableCols

cloudql/github/table_github_organization_external_identity.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@ func gitHubOrganizationExternalIdentityColumns() []*plugin.Column {
1717
Transform: transform.FromField("Description.UserLogin")},
1818
{Name: "user", Type: proto.ColumnType_JSON, Description: "The GitHub user details.",
1919
Transform: transform.FromField("Description.User")},
20+
{Name: "user_id", Type: proto.ColumnType_INT, Description: "The GitHub user details.",
21+
Transform: transform.FromField("Description.UserID")},
2022
{Name: "saml_identity", Type: proto.ColumnType_JSON, Description: "The external SAML identity.",
2123
Transform: transform.FromField("Description.SamlIdentity")},
2224
{Name: "scim_identity", Type: proto.ColumnType_JSON, Description: "The external SCIM identity.",

cloudql/github/table_github_organization_role.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -31,28 +31,28 @@ func gitHubOrganizationRoleColumns() []*plugin.Column {
3131
Transform: transform.FromField("Description.Permissions")},
3232
{
3333
Name: "organization",
34-
Type: proto.ColumnType_JSON,
35-
Description: "permissions",
34+
Type: proto.ColumnType_STRING,
35+
Description: "",
3636
Transform: transform.FromField("Description.Organization")},
3737
{
3838
Name: "source",
39-
Description: "permissions",
39+
Description: "",
4040
Type: proto.ColumnType_STRING,
4141
Transform: transform.FromField("Description.Source")},
4242
{
4343
Name: "base_role",
4444
Type: proto.ColumnType_STRING,
45-
Description: "permissions",
45+
Description: "",
4646
Transform: transform.FromField("Description.BaseRole")},
4747
{
4848
Name: "created_at",
4949
Type: proto.ColumnType_TIMESTAMP,
50-
Description: "permissions",
50+
Description: "",
5151
Transform: transform.FromField("Description.CreatedAt")},
5252
{
5353
Name: "updated_at",
5454
Type: proto.ColumnType_TIMESTAMP,
55-
Description: "permissions",
55+
Description: "",
5656
Transform: transform.FromField("Description.UpdatedAt")},
5757
}
5858

discovery/describers/organization_collaborators.go

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,11 @@ import (
1212
)
1313

1414
type CollaboratorEdge struct {
15-
Permission githubv4.RepositoryPermission `graphql:"permission @include(if:$includeOCPermission)" json:"permission"`
16-
Node steampipemodels.CollaboratorLogin `graphql:"node @include(if:$includeOCNode)" json:"node"`
15+
Permission githubv4.RepositoryPermission `graphql:"permission @include(if:$includeOCPermission)" json:"permission"`
16+
Node struct {
17+
Login githubv4.String `graphql:"login" json:"login"`
18+
ID githubv4.String `graphql:"id" json:"id"`
19+
} `graphql:"node @include(if:$includeOCNode)" json:"node"`
1720
}
1821

1922
func GetAllOrganizationsCollaborators(ctx context.Context, githubClient model.GitHubClient, organizationName string, stream *models.StreamSender) ([]models.Resource, error) {
@@ -76,7 +79,8 @@ func GetOrganizationCollaborators(ctx context.Context, githubClient model.GitHub
7679
Affiliation: "ALL",
7780
RepositoryName: node.Name,
7881
Permission: collaborator.Permission,
79-
UserLogin: collaborator.Node,
82+
UserLogin: string(collaborator.Node.Login),
83+
UserID: string(collaborator.Node.ID),
8084
},
8185
}
8286
if stream != nil {

discovery/describers/organization_external_identity.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ func GetOrganizationExternalIdentities(ctx context.Context, githubClient model.G
5555
OrganizationExternalIdentity: externalIdentity,
5656
Organization: org,
5757
UserLogin: externalIdentity.User.Login,
58+
UserID: externalIdentity.User.Id,
5859
},
5960
}
6061
if stream != nil {

discovery/describers/organization_role.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ func ListOrganizationRoles(ctx context.Context,
5151
ID: strconv.Itoa(r.ID),
5252
Name: r.Name,
5353
Description: model.OrganizationRoleDescription{
54-
Organization: r.Organization,
54+
Organization: organizationName,
5555
Name: r.Name,
5656
ID: r.ID,
5757
Source: r.Source,

discovery/pkg/es/resources_clients.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2571,6 +2571,7 @@ var listOrgCollaboratorsFilters = map[string]string{
25712571
"organization": "Description.Organization",
25722572
"permission": "Description.Permission",
25732573
"repository_name": "Description.RepositoryName",
2574+
"user_id": "Description.UserID",
25742575
"user_login": "Description.UserLogin",
25752576
}
25762577

@@ -2639,6 +2640,7 @@ var getOrgCollaboratorsFilters = map[string]string{
26392640
"organization": "Description.Organization",
26402641
"permission": "Description.Permission",
26412642
"repository_name": "Description.RepositoryName",
2643+
"user_id": "Description.UserID",
26422644
"user_login": "Description.UserLogin",
26432645
}
26442646

@@ -3021,6 +3023,7 @@ var listOrgExternalIdentityFilters = map[string]string{
30213023
"saml_identity": "Description.SamlIdentity",
30223024
"scim_identity": "Description.ScimIdentity",
30233025
"user": "Description.User",
3026+
"user_id": "Description.UserID",
30243027
"user_login": "Description.UserLogin",
30253028
}
30263029

@@ -3091,6 +3094,7 @@ var getOrgExternalIdentityFilters = map[string]string{
30913094
"saml_identity": "Description.SamlIdentity",
30923095
"scim_identity": "Description.ScimIdentity",
30933096
"user": "Description.User",
3097+
"user_id": "Description.UserID",
30943098
"user_login": "Description.UserLogin",
30953099
}
30963100

discovery/provider/model.go

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -765,7 +765,8 @@ type OrgCollaboratorsDescription struct {
765765
Affiliation string
766766
RepositoryName githubv4.String
767767
Permission githubv4.RepositoryPermission
768-
UserLogin steampipemodels.CollaboratorLogin
768+
UserLogin string
769+
UserID string
769770
}
770771

771772
type OrgAlertDependabotDescription struct {
@@ -801,6 +802,7 @@ type OrgExternalIdentityDescription struct {
801802
steampipemodels.OrganizationExternalIdentity
802803
Organization string
803804
UserLogin string
805+
UserID int
804806
}
805807

806808
type OrgMembersDescription struct {
@@ -1854,7 +1856,7 @@ type OrganizationRoleDescription struct {
18541856
Name string
18551857
Description string
18561858
Permissions []string
1857-
Organization interface{}
1859+
Organization string
18581860
Source string
18591861
BaseRole string
18601862
CreatedAt time.Time

0 commit comments

Comments
 (0)