Skip to content

security: narrow internal ingress CIDR (JIRA-4521) #521

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.

Sign up for GitHub

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

Open
dylanratcliffe wants to merge 1 commit into
base: main
Choose a base branch
from

security: narrow internal ingress CIDR (JIRA-4521)

6664c41
Select commit
Loading
Failed to load commit list.
Open

security: narrow internal ingress CIDR (JIRA-4521) #521

security: narrow internal ingress CIDR (JIRA-4521)
6664c41
Select commit
Loading
Failed to load commit list.
Reviews Test / Example / Annotations failed May 13, 2026 in 1s

Found 53 issues

Analysis complete

Found 53 issues across the changed files:

Level Count Blocks merge?
❌ failure 30 Yes (if check is required)
⚠️ warning 20 No
ℹ️ notice 3 No

Note: This check used 2 API update calls to upload all 53 annotations (GitHub limit: 50 per call).

Details

Annotation Field Reference

Field Type Required Notes
path string ✅ Repo-relative file path
start_line number ✅ 1-indexed
end_line number ✅ Same as start_line for single-line
start_column number ❌ 0-indexed; only when start_line === end_line
end_column number ❌ 0-indexed; only when start_line === end_line
annotation_level enum ✅ failure | warning | notice
message string ✅ Shown inline in the diff
title string ❌ Bold heading in annotation popup
raw_details string ❌ Expanded details in popup

Pagination

GitHub accepts at most 50 annotations per PATCH call. To upload more, call UpdateCheckRun multiple times in sequence:

const batches = chunk(annotations, 50);
for (let i = 0; i < batches.length; i++) {
  const isLast = i === batches.length - 1;
  await octokit.rest.checks.update({
    ...(isLast ? { status: 'completed', conclusion: 'failure' } : {}),
    output: { title, summary, annotations: batches[i] },
  });
}

Annotations

Check failure on line 12 in src/index.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Unused variable 

'connectionString' is declared but never read.
Raw output 
TypeScript error TS6133: 'connectionString' is declared but its value is never read.

Remove the variable or use it in the function body.

Check failure on line 52 in src/auth.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Hardcoded secret 

Secret key is hardcoded. Use environment variables or a secrets manager.
Raw output 
CWE-798: Use of Hard-coded Credentials

Replace with `process.env.SECRET_KEY` and add the variable to your secrets management system.

Check failure on line 3 in src/module-1.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-1.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 6 in src/module-2.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-2.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 9 in src/module-3.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-3.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 12 in src/module-4.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-4.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 15 in src/module-5.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-5.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 18 in src/module-6.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-6.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 21 in src/module-7.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-7.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 24 in src/module-8.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-8.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 27 in src/module-9.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-9.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 30 in src/module-10.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-10.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 33 in src/module-11.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-11.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 36 in src/module-12.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-12.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 39 in src/module-13.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-13.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 42 in src/module-14.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-14.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 45 in src/module-15.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-15.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 48 in src/module-16.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-16.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 51 in src/module-17.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-17.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 54 in src/module-18.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-18.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 57 in src/module-19.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-19.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 60 in src/module-20.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-20.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 63 in src/module-21.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-21.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 66 in src/module-22.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-22.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.

Check failure on line 69 in src/module-23.ts

See this annotation in the file changed.

@reviews-test reviews-test / Example / Annotations

Type error in module-23.ts 

Argument of type 'string' is not assignable to parameter of type 'number'.
View more details on Reviews Test