-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[docdb] coredump in YSQL webserver #7683
Labels
area/docdb
YugabyteDB core features
area/ysql
Yugabyte SQL (YSQL)
kind/bug
This issue is a bug
priority/low
Low priority
Comments
bmatican
added
area/ysql
Yugabyte SQL (YSQL)
area/docdb
YugabyteDB core features
labels
Mar 16, 2021
notes from Sergei
|
yugabyte-ci
added
kind/bug
This issue is a bug
priority/medium
Medium priority issue
labels
Jun 8, 2022
@es1024 Passing this to you, since you're looking at the core webserver code as well. This is lower pri though. |
bmatican
added
priority/low
Low priority
and removed
priority/medium
Medium priority issue
labels
Jul 19, 2022
Resolved with f2702bb |
samiahmedsiddiqui
pushed a commit
to samiahmedsiddiqui/yugabyte-db
that referenced
this issue
Aug 16, 2022
…crash when enabling SSL Summary: This diff fixes a crash when enable SSL for the webserver, due to an old version of Squeasel: 1. searching for and loading OpenSSL from the system rather than using the one bundled with thirdparty. 2. depending on the dynamic library exposing symbols that have been removed with OpenSSL 1.1.0 (SSLv23_... methods, which have been replaced with a macro). This is fixed by rebasing to a newer upstream commit, which does not require those symbols, and which further presents the ssl_global_init flag to stop Squeasel from loading OpenSSL on its own. This diff also adds the `webserver_private_key_file` and `webserver_private_key_password` gflags, corresponding to the `ssl_private_key` and `ssl_private_key_password` options present in newer versions of Squeasel. This diff also fixes a crash when pthread_setspecific fails in libcds, by catching the exception and forcing the worker thread to exit instead of letting it bubble up and kill the whole process. Also pulling yugabyte/yugabyte-db-thirdparty@fc8dae1 to fix C++ exception handling in Clang-based LTO builds (yugabyte#13064). Test Plan: `ybd --cxx_test server_webserver-test --gtest_filter WebserverSecureTest.TestIndexPage` Reviewers: mbautin Reviewed By: mbautin Subscribers: cwang, sanketh, ybase, bogdan Differential Revision: https://phabricator.dev.yugabyte.com/D18419
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area/docdb
YugabyteDB core features
area/ysql
Yugabyte SQL (YSQL)
kind/bug
This issue is a bug
priority/low
Low priority
Jira Link: DB-1734
Not much to go on here, these were all the 3 threads in the coredump:
Running a small YSQL workload on a 2DC enabled cluster, but that shouldn't be much related to the YSQL webserver..
@mbautin @spolitov you're probably more familiar with the cds::threading piece, any thoughts?
The text was updated successfully, but these errors were encountered: