Skip to content

[java] Allowing setting SSL context in client config for HttpClient#12874

Merged
titusfortner merged 4 commits intoSeleniumHQ:trunkfrom
pujagani:add-sslcontext-to-clientconfig
Oct 6, 2023
Merged

[java] Allowing setting SSL context in client config for HttpClient#12874
titusfortner merged 4 commits intoSeleniumHQ:trunkfrom
pujagani:add-sslcontext-to-clientconfig

Conversation

@pujagani
Copy link
Contributor

@pujagani pujagani commented Oct 5, 2023

Thanks for contributing to Selenium!
A PR well described will help maintainers to quickly review and merge it

Before submitting your PR, please check our contributing guidelines.
Avoid large PRs, help reviewers by making them as simple and short as possible.

Description

Allowing setting SSL context in client config for HttpClient

Motivation and Context

Fixes #12869

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I have read the contributing document.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@pujagani pujagani added the C-java Java Bindings label Oct 5, 2023
@codecov-commenter
Copy link

codecov-commenter commented Oct 5, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (d13ac33) 56.51% compared to head (e78fd13) 56.51%.
Report is 3 commits behind head on trunk.

❗ Current head e78fd13 differs from pull request most recent head 207edc6. Consider uploading reports for the commit 207edc6 to get more accurate results

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##            trunk   #12874   +/-   ##
=======================================
  Coverage   56.51%   56.51%           
=======================================
  Files          86       86           
  Lines        5255     5255           
  Branches      187      187           
=======================================
  Hits         2970     2970           
  Misses       2098     2098           
  Partials      187      187           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@titusfortner titusfortner merged commit b9bdff1 into SeleniumHQ:trunk Oct 6, 2023
aguspe pushed a commit to aguspe/selenium that referenced this pull request Oct 22, 2023
…eleniumHQ#12874)

* [java] Allowing setting SSL context in client config for HttpClient

Fixes SeleniumHQ#12869

* [java] Fix refactor error

* fix linter complaints

---------

Co-authored-by: titusfortner <titus.fortner@gmail.com>
Co-authored-by: Titus Fortner <titusfortner@users.noreply.github.com>
aguspe pushed a commit to aguspe/selenium that referenced this pull request Oct 29, 2023
…eleniumHQ#12874)

* [java] Allowing setting SSL context in client config for HttpClient

Fixes SeleniumHQ#12869

* [java] Fix refactor error

* fix linter complaints

---------

Co-authored-by: titusfortner <titus.fortner@gmail.com>
Co-authored-by: Titus Fortner <titusfortner@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-java Java Bindings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🐛 Bug]: SSLHandshakeException for JDK Http Client

4 participants