Skip to content

Addressing 'org.apache.hc.core5.http.ParseException: Invalid protocol version' under JDK 16+#287

Merged
lezzago merged 1 commit intoopensearch-project:mainfrom
reta:fix.jdk.17.tls.factory
Jan 20, 2023
Merged

Addressing 'org.apache.hc.core5.http.ParseException: Invalid protocol version' under JDK 16+#287
lezzago merged 1 commit intoopensearch-project:mainfrom
reta:fix.jdk.17.tls.factory

Conversation

@reta
Copy link
Contributor

@reta reta commented Oct 18, 2022

Signed-off-by: Andriy Redko andriy.redko@aiven.io

Description

When using JDK 16+, the org.apache.hc.core5.http.ParseException: Invalid protocol version is being thrown.

Issues Resolved

See please https://issues.apache.org/jira/browse/HTTPCLIENT-2219

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@reta reta requested a review from a team October 18, 2022 18:49
@codecov-commenter
Copy link

Codecov Report

Merging #287 (6797c20) into main (c231644) will increase coverage by 0.08%.
The diff coverage is 87.87%.

@@             Coverage Diff              @@
##               main     #287      +/-   ##
============================================
+ Coverage     71.52%   71.60%   +0.08%     
- Complexity      666      667       +1     
============================================
  Files           108      108              
  Lines          4477     4494      +17     
  Branches        605      605              
============================================
+ Hits           3202     3218      +16     
- Misses         1024     1025       +1     
  Partials        251      251              
Impacted Files Coverage Δ
...ain/java/org/opensearch/commons/authuser/User.java 87.82% <ø> (ø)
...commons/destination/message/LegacyBaseMessage.java 67.44% <ø> (ø)
...estination/message/LegacyCustomWebhookMessage.java 74.68% <ø> (ø)
...ensearch/commons/rest/SecureRestClientBuilder.java 53.15% <75.00%> (+2.66%) ⬆️
...search/commons/alerting/action/GetAlertsRequest.kt 97.14% <100.00%> (+0.47%) ⬆️
...arch/commons/alerting/model/ClusterMetricsInput.kt 80.00% <100.00%> (+0.51%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@reta
Copy link
Contributor Author

reta commented Dec 15, 2022

@dblock could you please check this one out?

… version' under JDK 16+

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
@dblock
Copy link
Member

dblock commented Jan 18, 2023

Sorry for the delay, just noticing this. Why didn't CI run here?

@reta
Copy link
Contributor Author

reta commented Jan 18, 2023

Sorry for the delay, just noticing this. Why didn't CI run here?

Hm .. not sure to be fair, the run has to be approved for sure

@dblock
Copy link
Member

dblock commented Jan 18, 2023

I have admin rights on the org and yet I don't see any approval button ... let me force push something here.

@dblock dblock mentioned this pull request Jan 18, 2023
@dblock
Copy link
Member

dblock commented Jan 18, 2023

Looks like CI kicked in now 🤷

@dblock dblock requested a review from lezzago January 18, 2023 23:26
@reta
Copy link
Contributor Author

reta commented Jan 18, 2023

Looks like CI kicked in now 🤷

Thanks a lot @dblock !

@dblock dblock requested a review from VachaShah January 19, 2023 16:00
@lezzago lezzago merged commit 4ef49e5 into opensearch-project:main Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants