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
V3.0.x cdp #4992
base: v3.0.x
Are you sure you want to change the base?
V3.0.x cdp #4992
Conversation
natarajanmm
commented
May 11, 2023
…ephemeral child were actual processing happens) unlike UDP. Stats need to update parent incase of TCP and also find socket to update didn't look for TCP sockets. Addressed both these.
…ilable as an attribute helps in certain use cases in python plugin, currently this attribute from certificate isn't parsesd. Added support to parse this from the certificate.
src/main/tls.c
Outdated
if (crl_dp != NULL) { | ||
for (int i = 0; i < sk_DIST_POINT_num(crl_dp); i++) { | ||
dp = sk_DIST_POINT_value(crl_dp, i); | ||
if (dp != NULL) { |
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.
Too many nested if's. Consider using continue
when dp == NULL
src/main/tls.c
Outdated
if (dp != NULL) { | ||
cdp[0] = '\0'; | ||
url_ptr = get_dp_url(dp); | ||
if (url_ptr != NULL) { |
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.
continue #2
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.
Addressed both the comments
We won't merge this into v3.0.x. That branch is for security issues and bug fixes only. There are no new features in 3.0. We will take a look at adding these features in 3.2. |