-
Notifications
You must be signed in to change notification settings - Fork 301
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
HPCC-20631 Set session id context #11764
HPCC-20631 Set session id context #11764
Conversation
https://track.hpccsystems.com/browse/HPCC-20631 |
@mckellyln please review. |
@@ -896,6 +900,9 @@ class CSecureSocketContext : implements ISecureSocketContext, public CInterface | |||
{ | |||
throw MakeStringException(-1, "ctx can't be created"); | |||
} | |||
|
|||
setSessionIdContext(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think setting the ctx session is meaningful only on the server side ?
If that is true, should we do this only for server, ie:
if(sockettype == ServerSocket)
setSessionIdContext();
Or do this only before an SSL_accept() ?
@mayx a few questions: |
|
@mayx ok, thanks - so should it just be done before SSL_accept() or only if sockettype == ServerSocket ? |
Good point. The man page states it's only "useful" for server side and it doesn't seem to affect client side functioning. But to stay on the safe side, I'll add the check of sockettype. |
- Set the session id context of an ssl context to be "hpccsystems", a value unique to our applicaiton. Signed-off-by: mayx <yanrui.ma@lexisnexisrisk.com>
dd2b1e8
to
4a1f1dc
Compare
@mckellyln change uploaded please review again. |
Automated Smoketest: ✅ Unit tests result:
Regression test result:
HPCC Stop: OK
|
@mayx looks good. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mayx approved
value unique to our applicaiton.
Signed-off-by: mayx yanrui.ma@lexisnexisrisk.com
Type of change:
Checklist:
Testing: