[SERVER-37993] Upgrade asio to eed287d46c14310f0daf4ff19b0297917143723 Created: 07/Nov/18  Updated: 29/Oct/23  Resolved: 17/Dec/18

Status: Closed
Project: Core Server
Component/s: Internal Code
Affects Version/s: None
Fix Version/s: 4.1.7, 4.0.20

Type: Improvement Priority: Major - P3
Reporter: Mathew Robinson (Inactive) Assignee: Andrew Morrow (Inactive)
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Backports
Depends
Problem/Incident
causes SERVER-79022 Update ASIO's Git hash in README.thir... Closed
Backwards Compatibility: Fully Compatible
Backport Requested:
v4.0
Sprint: Dev Tools 2018-12-31
Participants:
Linked BF Score: 0

 Comments   
Comment by Githook User [ 24/Jul/20 ]

Author:

{'name': 'Andrew Morrow', 'email': 'acm@mongodb.com', 'username': 'acmorrow'}

Message: SERVER-37993 Upgrade ASIO to 1.12.2
Branch: v4.0
https://github.com/mongodb/mongo/commit/4f97328b8d3f910b27190e86ea7ea27f7f3e372a

Comment by Githook User [ 17/Dec/18 ]

Author:

{'username': 'acmorrow', 'email': 'acm@mongodb.com', 'name': 'Andrew Morrow'}

Message: SERVER-37993 Upgrade ASIO to 1.12.2
Branch: master
https://github.com/mongodb/mongo/commit/0c6f72ea1828b9dbbcc86e61d04e1f925ca870fc

Comment by Andrew Morrow (Inactive) [ 08/Nov/18 ]

OK, thanks for clarifying.

Comment by Mark Benvenuto [ 07/Nov/18 ]

There is no changes needed to our forked copy unless in the unlikely event it stops compiling.

The forked copy only forks the asio::stream and asio::context classes. The stream class is just responsible for shipping bytes from OpenSSL BIO to asio and managing state. The context class just holds a handle to OpenSSL BIO. We do not depend on ASIO for SSL context setup for instance. Things like TLS version, certificate management and other setup is all done in ssl_manager_openssl.cpp.

Furthermore, when I copied these classes over, I stripped them as much as possible to minimize the amount of maintenance we will have to do going forward. Unless ASIO changes their API in some significant way or we move off ASIO, I don't foresee a need for lots of changes even as we update ASIO.

This means that changes like https://github.com/chriskohlhoff/asio/commit/42a94f7cb4828139542042354a8893058fe1272f are irrelevant for us since we manage which version of TLS in ssl_manager_openssl.cpp and it has no affect on our TLS version support.

Comment by Andrew Morrow (Inactive) [ 07/Nov/18 ]

Shouldn't we be looking to move that forward as well?

Comment by Mark Benvenuto [ 07/Nov/18 ]

Neither Sara nor I have made any modifications to ASIO in asio-master. We have a forked copy of the ssl library in src/mongo/util/net/ssl but this will likely not affect you.

Generated at Thu Feb 08 04:47:39 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.