Skip to content
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.

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

failing to build with cryptography 3.1 #234

Closed
dirk-thomas opened this issue Aug 27, 2020 · 4 comments · Fixed by ros2/ci#506
Closed

failing to build with cryptography 3.1 #234

dirk-thomas opened this issue Aug 27, 2020 · 4 comments · Fixed by ros2/ci#506
Assignees
Labels
bug Something isn't working

Comments

@dirk-thomas
Copy link
Member

The nightly ROS 2 builds failed on macOS and Windows. I assume due to the new cryptography version 3.1 being released.

@dirk-thomas dirk-thomas added the bug Something isn't working label Aug 27, 2020
@dirk-thomas
Copy link
Member Author

Upstream ticket: pyca/cryptography#5433.

@kyrofa
Copy link
Member

kyrofa commented Aug 27, 2020

Oh that's unfortunate. I see three paths forward:

  1. Wait for cryptography to add a proper API to replace the bindings
  2. Move back to shelling out to openssl directly
  3. Pin to cryptography 3.0

I think (1) is really the only real solution here-- the others are workarounds and temporary solutions at best. They at least seem interested in exposing this via the API though, maybe we can help. We'll push on that a little, it looks like you've started on (3) for now?

@dirk-thomas
Copy link
Member Author

The referenced PR implements option 3 to get the jobs passing now.

Upstream is working on re-adding the necessary API as described in option 1 (see discussion on the upstream ticket). If that has been released we could unpin (which is tracked in ros2/ci#507).

@kyrofa
Copy link
Member

kyrofa commented Aug 27, 2020

Excellent, thanks @dirk-thomas, sounds like a plan. We'll join the upstream discussion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants