Bug 874023 - [PATCH] (lib)curl ignores OPENSSL_CONF environment variable
Summary: [PATCH] (lib)curl ignores OPENSSL_CONF environment variable
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: curl
Version: rawhide
Hardware: All
OS: All
unspecified
low
Target Milestone: ---
Assignee: Kamil Dudka
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-11-07 10:19 UTC by Artem S. Tashkinov
Modified: 2012-11-07 21:54 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2012-11-07 21:02:55 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
A patch for ssluse.c to make use of OPENSSL_CONF environment variable (443 bytes, patch)
2012-11-07 10:19 UTC, Artem S. Tashkinov
no flags Details | Diff

Description Artem S. Tashkinov 2012-11-07 10:19:52 UTC
Created attachment 639924 [details]
A patch for ssluse.c to make use of OPENSSL_CONF environment variable

OpenSSL documentation says that you can use  OPENSSL_CONF environment variable to redefine openssl configuration file.

CURL by default ignores this variable.

This patch fixes this problem:

Comment 1 Kamil Dudka 2012-11-07 21:02:55 UTC
Thank you for contributing the patch, however we do not build libcurl against OpenSSL in Fedora packages, so your patch is not applicable.  Please have a look at NSS environment variables:

https://developer.mozilla.org/en-US/docs/NSS_reference/NSS_environment_variables

Comment 2 Artem S. Tashkinov 2012-11-07 21:20:02 UTC
NSS doesn't support GOST encryption standards and other openssl features but, I guess, fedora and redhat people know what's better for me.

In the meanwhile I will compile curl with openssl on my system.

Comment 3 Kamil Dudka 2012-11-07 21:41:14 UTC
You can consider submitting your patch upstream.  It will not affect libcurl packages on Fedora, but you can get some feedback from users of OpenSSL-powered libcurl:

http://curl.haxx.se/docs/faq.html#How_do_I_submit_my_patch

... and chances are the patch will be accepted there.

Comment 4 Artem S. Tashkinov 2012-11-07 21:54:31 UTC
You're quite right, this patch should be merged upstream.


Note You need to log in before you can comment on or make changes to this bug.