Skip to content

HTTP request callback support#1689

Merged
SteveSandersonMS merged 56 commits into
mainfrom
stevesandersonms/llm-inference-callbacks
Jun 23, 2026
Merged

HTTP request callback support#1689
SteveSandersonMS merged 56 commits into
mainfrom
stevesandersonms/llm-inference-callbacks

Add cancel + error e2e coverage for CopilotRequestHandler (Node + .NET)

692eb2c
Select commit
Loading
Failed to load commit list.
GitHub Advanced Security / CodeQL failed Jun 22, 2026 in 3s

46 new alerts including 1 high severity security vulnerability

New alerts in code changed by this pull request

Security Alerts:

  • 1 high

Other Alerts:

  • 2 warnings
  • 43 notes

See annotations below for details.

View all branch alerts.

Annotations

Check notice on line 530 in rust/tests/e2e/llm_inference.rs

See this annotation in the file changed.

Code scanning / CodeQL

Unused variable Note test

Variable 'session_id' is not used.

Check notice on line 569 in rust/tests/e2e/llm_inference.rs

See this annotation in the file changed.

Code scanning / CodeQL

Unused variable Note test

Variable 'session_id' is not used.

Check notice on line 667 in rust/tests/e2e/llm_inference.rs

See this annotation in the file changed.

Code scanning / CodeQL

Unused variable Note test

Variable 'err' is not used.

Check notice on line 828 in rust/tests/e2e/llm_inference.rs

See this annotation in the file changed.

Code scanning / CodeQL

Unused variable Note test

Variable 'err' is not used.

Check failure on line 160 in java/src/test/java/com/github/copilot/FakeUpstreamServer.java

See this annotation in the file changed.

Code scanning / CodeQL

Use of a potentially broken or risky cryptographic algorithm High test

Cryptographic algorithm
SHA-1
Loading
may not be secure. Consider using a different algorithm.

Check notice on line 62 in java/src/main/java/com/github/copilot/ForwardingWebSocketHandler.java

See this annotation in the file changed.

Code scanning / CodeQL

Useless parameter Note

The parameter 'binary' is never used.

Check notice on line 76 in java/src/main/java/com/github/copilot/ForwardingWebSocketHandler.java

See this annotation in the file changed.

Code scanning / CodeQL

Useless parameter Note

The parameter 'binary' is never used.

Check notice on line 241 in java/src/main/java/com/github/copilot/LlmInferenceAdapter.java

See this annotation in the file changed.

Code scanning / CodeQL

Deprecated method or constructor invocation Note

Invoking
JsonNode.fields
Loading
should be avoided because it has been deprecated.

Check notice on line 120 in java/src/test/java/com/github/copilot/FakeUpstreamServer.java

See this annotation in the file changed.

Code scanning / CodeQL

Missing catch of NumberFormatException Note test

Potential uncaught 'java.lang.NumberFormatException'.

Check warning on line 54 in python/e2e/test_llm_inference_websocket_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Signature mismatch in overriding method Warning test

This method requires 2 positional arguments, whereas overridden
LlmRequestHandler._handle_web_socket
Loading
requires 3.
This call
Loading
correctly calls the base method, but does not match the signature of the overriding method.

Check notice on line 62 in python/e2e/test_llm_inference_consumer_cancel_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Except block handles 'BaseException' Note test

Except block directly handles BaseException.

Check notice on line 19 in python/e2e/test_llm_inference_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Unused import Note test

Import of 'E2ETestContext' is not used.

Check notice on line 57 in python/e2e/test_llm_inference_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Empty except Note test

'except' clause does nothing but pass and there is no explanatory comment.

Check notice on line 24 in python/e2e/test_llm_inference_errors_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Unused import Note test

Import of 'E2ETestContext' is not used.

Check notice on line 63 in python/e2e/test_llm_inference_errors_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Except block handles 'BaseException' Note test

Except block directly handles BaseException.

Check notice on line 20 in python/e2e/test_llm_inference_handler_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Unused import Note test

Import of 'asyncio' is not used.

Check notice on line 30 in python/e2e/test_llm_inference_handler_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Unused import Note test

Import of 'websockets' is not used.

Check notice on line 239 in python/e2e/test_llm_inference_handler_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Empty except Note test

'except' clause does nothing but pass and there is no explanatory comment.

Check notice on line 26 in python/e2e/test_llm_inference_session_id_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Unused import Note test

Import of 'E2ETestContext' is not used.

Check notice on line 24 in python/e2e/test_llm_inference_stream_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Unused import Note test

Import of 'E2ETestContext' is not used.

Check notice on line 30 in python/e2e/test_llm_inference_websocket_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Unused import Note test

Import of 'E2ETestContext' is not used.

Check notice on line 23 in python/e2e/test_llm_inference_consumer_cancel_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Unused import Note test

Import of 'E2ETestContext' is not used.

Check notice on line 25 in python/e2e/test_llm_inference_cancel_e2e.py

See this annotation in the file changed.

Code scanning / CodeQL

Unused import Note test

Import of 'E2ETestContext' is not used.

Check notice on line 317 in python/e2e/_llm_inference_helpers.py

See this annotation in the file changed.

Code scanning / CodeQL

Empty except Note

'except' clause does nothing but pass and there is no explanatory comment.

Check notice on line 307 in python/copilot/llm_request_handler.py

See this annotation in the file changed.

Code scanning / CodeQL

Empty except Note

'except' clause does nothing but pass and there is no explanatory comment.