https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
Mark Thomas changed:
What|Removed |Added
Status|REOPENED|RESOLVED
Resolution|---
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
Michael Osipov changed:
What|Removed |Added
CC||micha...@apache.org
--- Comment #15 f
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
--- Comment #14 from Edin Hodzic ---
One other potential improvement in the OpenSSL interaction might be to enable
OpenSSL configuration:
In ssl.c, initialization is done with the call:
OPENSSL_init_ssl(OPENSSL_INIT_ENGINE_ALL_BUILTIN, NUL
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
Edin Hodzic changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|FIXED
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
Edin Hodzic changed:
What|Removed |Added
Attachment #37870|0 |1
is obsolete|
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
--- Comment #12 from Edin Hodzic ---
Comment on attachment 37870
--> https://bz.apache.org/bugzilla/attachment.cgi?id=37870
keep structural reference to the SSL Engine pointed to by tcn_ssl_engine
diff -aur tomcat-native-1.2.28-src/native/sr
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
--- Comment #11 from Edin Hodzic ---
Created attachment 37870
--> https://bz.apache.org/bugzilla/attachment.cgi?id=37870&action=edit
keep structural reference to the SSL Engine pointed to by tcn_ssl_engine
Found another problem with the rece
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
Mark Thomas changed:
What|Removed |Added
Status|REOPENED|RESOLVED
Resolution|---
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
--- Comment #9 from Edin Hodzic ---
> As such, the code above has to be
"&& (tcn_ssl_engine == NULL || ENGINE_load_private_key() fails)".
I agree, my bad on the original patch. :/
--
You are receiving this mail because:
You are the assignee
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
Konstantin Kolinko changed:
What|Removed |Added
Resolution|FIXED |---
Status|RESOLVED
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
--- Comment #7 from Edin Hodzic ---
Thanks everyone for the prompt action and the pending change.
--
You are receiving this mail because:
You are the assignee for the bug.
-
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
Mark Thomas changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
--- Comment #5 from mgrigorov ---
(In reply to Edin Hodzic from comment #4)
> Could I submit a pull request somehow?
https://github.com/apache/tomcat-native
--
You are receiving this mail because:
You are the assignee for the bug.
--
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
--- Comment #4 from Edin Hodzic ---
Thanks for the updates, the review and the comments.
The NULL comparison I can add.
The change was tested with and without the SSL engine in the configuration.
Is there anything else I can do to get the pa
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
--- Comment #3 from Christopher Schultz ---
A quick review:
>...
>&& tcn_ssl_engine &&
>(c->keys[idx] = ENGINE_load_private_key(tcn_ssl_engine, key_file,
>NULL, NULL)) == NULL
This will
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
Christopher Schultz changed:
What|Removed |Added
Severity|major |enhancement
--- Comment #2 from
https://bz.apache.org/bugzilla/show_bug.cgi?id=65181
Edin Hodzic changed:
What|Removed |Added
Severity|enhancement |major
Priority|P2
17 matches
Mail list logo