Skip to content

Commit

Permalink
Remove obsolete comment
Browse files Browse the repository at this point in the history
Co-authored-by: Marco A. Gutierrez <[email protected]>
Signed-off-by: Mikael Arguedas <[email protected]>
  • Loading branch information
mikaelarguedas and marcoag authored May 7, 2024
1 parent c7119d5 commit 5006e37
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
3 changes: 0 additions & 3 deletions sros2/sros2/_utilities.py
Original file line number Diff line number Diff line change
Expand Up @@ -92,9 +92,6 @@ def build_key_and_cert(subject_name, *, ca=False, ca_key=None, issuer_name=''):
).serial_number(
x509.random_serial_number()
).not_valid_before(
# Using a day earlier here to prevent Connext (5.3.1) from complaining
# when extracting it from the permissions file and thinking it's in the future
# https://github.com/ros2/ci/pull/436#issuecomment-624874296
utcnow
).not_valid_after(
# TODO: This should not be hard-coded
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -125,9 +125,6 @@ def test_cert_pem(enclave_keys_dir):
# Verify the cert is valid for the expected timespan
utcnow = datetime.datetime.now(datetime.UTC)

# Using a day earlier here to prevent Connext (5.3.1) from complaining
# when extracting it from the permissions file and thinking it's in the future
# https://github.com/ros2/ci/pull/436#issuecomment-624874296
# TODO replace "not_valid_before"/"not_valid_after" functions by
# "not_valid_before_utc"/"not_valid_after_utc"
# once cryptography 42 is supported on all target platforms
Expand Down

0 comments on commit 5006e37

Please sign in to comment.