Skip to content

Commit 0bd9fa8

Browse files
author
Evan Gibler
authored
Update GitHub Data Model to display admin-add events instead of UNKNOWN_ROLE (#979)
1 parent 137d2a8 commit 0bd9fa8

File tree

2 files changed

+38
-5
lines changed

2 files changed

+38
-5
lines changed

data_models/github_data_model.py

+11-5
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,20 @@
11
import panther_event_type_helpers as event_type
22

3+
ADMIN_EVENTS = {
4+
"business.add_admin",
5+
"business.invite_admin",
6+
"team.promote_maintainer",
7+
}
38

4-
def get_admin_role(_):
5-
# github doesn't record the admin role in the event
6-
return "<UNKNOWN_ROLE>"
9+
10+
def get_admin_role(event):
11+
action = event.get("action", "")
12+
return action if action in ADMIN_EVENTS else "<UNKNOWN_ADMIN_ROLE>"
713

814

915
def get_event_type(event):
10-
if event.get("action") == "team.promote_maintainer":
16+
if event.get("action", "") in ADMIN_EVENTS:
1117
return event_type.ADMIN_ROLE_ASSIGNED
12-
if event.get("action") == "org.disable_two_factor_requirement":
18+
if event.get("action", "") == "org.disable_two_factor_requirement":
1319
return event_type.MFA_DISABLED
1420
return None

rules/standard_rules/admin_assigned.yml

+27
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,33 @@ Tests:
166166
"p_log_type": "GitHub.Audit",
167167
"user": "bob"
168168
}
169+
- Name: Github - Admin Added
170+
ExpectedResult: true
171+
Log:
172+
{
173+
"actor": "cat",
174+
"action": "business.add_admin",
175+
"p_log_type": "GitHub.Audit",
176+
"user": "bob"
177+
}
178+
- Name: Github - Admin Invited
179+
ExpectedResult: true
180+
Log:
181+
{
182+
"actor": "cat",
183+
"action": "business.invite_admin",
184+
"p_log_type": "GitHub.Audit",
185+
"user": "bob"
186+
}
187+
- Name: Github - Unknown Admin Role
188+
ExpectedResult: false
189+
Log:
190+
{
191+
"actor": "cat",
192+
"action": "unknown.admin_role",
193+
"p_log_type": "GitHub.Audit",
194+
"user": "bob"
195+
}
169196
-
170197
Name: Zendesk - Admin Role Downgraded
171198
ExpectedResult: false

0 commit comments

Comments
 (0)