Skip to content

Pre-2.9#172

Merged
cwlacewe merged 16 commits intodevelopfrom
pre-2.9
Jul 24, 2024
Merged

Pre-2.9#172
cwlacewe merged 16 commits intodevelopfrom
pre-2.9

Conversation

@cwlacewe
Copy link
Copy Markdown
Contributor

@cwlacewe cwlacewe commented Jul 22, 2024

This PR contains code for release 2.9.0 which includes:

  • mutual TLS support for C++ client
  • Timing capability to allow users to evaluate the performance
  • Ability to dynamically add metadata from videos by leveraging the UDF capability
  • Improvements in Neo4j Handler’s error handling and reporting
  • Updates to dependencies/Dockerfiles to address vulnerabilities

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 22, 2024

Target CPP Coverage: %
Source CPP Coverage: 62.0326%

Target Python Coverage: %
Source Python Coverage: 98%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 22, 2024

Target CPP Coverage: 0.00%
Source CPP Coverage: 62.0326%

Target Python Coverage: 0.00%
Source Python Coverage: 98%

github-advanced-security[bot]
github-advanced-security bot found potential problems Jul 22, 2024
View reviewed changes
tests/tls_test/run_tls_test_client.py
for _ in range(loops):
conn, addr = sock.accept()
with conn:
with context.wrap_socket(conn, server_side=True) as ssock:

Check failure

Code scanning / CodeQL

Use of insecure SSL/TLS version

Insecure SSL/TLS protocol version TLSv1 allowed by [call to ssl.create_default_context](1). Insecure SSL/TLS protocol version TLSv1_1 allowed by [call to ssl.create_default_context](1).
tests/tls_test/run_tls_test_server.py
with socket.create_connection(
(self.host, self.port), timeout=self.timeout
) as sock:
with context.wrap_socket(sock, server_hostname=self.host) as ssock:

Check failure

Code scanning / CodeQL

Use of insecure SSL/TLS version

Insecure SSL/TLS protocol version TLSv1 allowed by [call to ssl.create_default_context](1). Insecure SSL/TLS protocol version TLSv1_1 allowed by [call to ssl.create_default_context](1).
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 23, 2024

Target CPP Coverage: 0.00%
Source CPP Coverage: 64.4319%

Target Python Coverage: 0.00%
Source Python Coverage: 98%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 23, 2024

Target CPP Coverage: 0.00%
Source CPP Coverage: 64.4418%

Target Python Coverage: 0.00%
Source Python Coverage: 47%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 23, 2024

Target CPP Coverage: 0.00%
Source CPP Coverage: 64.4418%

Target Python Coverage: 0.00%
Source Python Coverage: 98%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 23, 2024

Target CPP Coverage: 0.00%
Source CPP Coverage: 64.4418%

Target Python Coverage: 0.00%
Source Python Coverage: 98%

@cwlacewe cwlacewe marked this pull request as ready for review July 24, 2024 15:40
rolandoquesada
rolandoquesada approved these changes Jul 24, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor

@rolandoquesada rolandoquesada left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 24, 2024

Target CPP Coverage: 0.00%
Source CPP Coverage: 64.4418%

Target Python Coverage: 0.00%
Source Python Coverage: 98%

@cwlacewe cwlacewe merged commit 6b80c68 into develop Jul 24, 2024
@cwlacewe cwlacewe deleted the pre-2.9 branch July 24, 2024 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@keirafadams keirafadams Awaiting requested review from keirafadams

1 more reviewer

@rolandoquesada rolandoquesada rolandoquesada approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@cwlacewe @github-advanced-security @rolandoquesada @sys-vdms