Skip to content

Commit 876750e

Browse files
alexr00Copilot
andcommitted
Also for assignable users
Co-authored-by: Copilot <copilot@github.com>
1 parent 828630b commit 876750e

1 file changed

Lines changed: 21 additions & 14 deletions

File tree

src/github/githubRepository.ts

Lines changed: 21 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -144,6 +144,21 @@ export function isRateLimitError(e: unknown): boolean {
144144
return false;
145145
}
146146

147+
export function getErrorCode(e: any): string | undefined {
148+
if (e.status !== undefined) {
149+
return String(e.status);
150+
} else if (e.networkError?.statusCode !== undefined) {
151+
return String(e.networkError.statusCode);
152+
} else if (e.graphQLErrors?.[0]?.extensions?.code) {
153+
return String(e.graphQLErrors[0].extensions.code);
154+
} else if (e.code !== undefined) {
155+
return String(e.code);
156+
} else if (e.name) {
157+
return e.name;
158+
}
159+
return undefined;
160+
}
161+
147162
export enum TeamReviewerRefreshKind {
148163
None,
149164
Try,
@@ -1255,16 +1270,9 @@ export class GitHubRepository extends Disposable {
12551270
succeededInOtherRepo: String(succeededInOtherRepo),
12561271
callerName
12571272
};
1258-
if (e.status !== undefined) {
1259-
properties.errorCode = String(e.status);
1260-
} else if (e.networkError?.statusCode !== undefined) {
1261-
properties.errorCode = String(e.networkError.statusCode);
1262-
} else if (e.graphQLErrors?.[0]?.extensions?.code) {
1263-
properties.errorCode = String(e.graphQLErrors[0].extensions.code);
1264-
} else if (e.code !== undefined) {
1265-
properties.errorCode = String(e.code);
1266-
} else if (e.name) {
1267-
properties.errorCode = e.name;
1273+
const errorCode = getErrorCode(e);
1274+
if (errorCode) {
1275+
properties.errorCode = errorCode;
12681276
}
12691277
/* __GDPR__
12701278
"pr.getPullRequestFailed" : {
@@ -1587,10 +1595,9 @@ export class GitHubRepository extends Disposable {
15871595
const properties: { errorCode?: string; usedSuggestedActors: string } = {
15881596
usedSuggestedActors: String(!!schema.GetSuggestedActors),
15891597
};
1590-
if (e.status !== undefined) {
1591-
properties.errorCode = String(e.status);
1592-
} else if (e.graphQLErrors?.[0]?.extensions?.code) {
1593-
properties.errorCode = String(e.graphQLErrors[0].extensions.code);
1598+
const errorCode = getErrorCode(e);
1599+
if (errorCode) {
1600+
properties.errorCode = errorCode;
15941601
}
15951602
/* __GDPR__
15961603
"pr.getAssignableUsersFailed" : {

0 commit comments

Comments
 (0)