Skip to content

Add support for verify peer.#15834

Merged
conniey merged 17 commits intoAzure:masterfrom
conniey:feature/verify-peer
Oct 2, 2020
Merged

Add support for verify peer.#15834
conniey merged 17 commits intoAzure:masterfrom
conniey:feature/verify-peer

Conversation

@conniey
Copy link
Member

@conniey conniey commented Sep 30, 2020

  • Add logic to ConnectionHandler and support for verification options.
  • Ensure that ClientOptions is not null.
  • Removing MockReactorProviderHandler because you can mock concrete classes.

@conniey conniey added Event Hubs Service Bus Client This issue points to a problem in the data-plane of the library. Azure.Core azure-core labels Sep 30, 2020
@conniey conniey self-assigned this Sep 30, 2020
@conniey
Copy link
Member Author

conniey commented Sep 30, 2020

/azp run java - eventhubs -tests

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@conniey
Copy link
Member Author

conniey commented Sep 30, 2020

/azp run java - eventhubs - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@conniey
Copy link
Member Author

conniey commented Sep 30, 2020

/azp run java - eventhubs - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@conniey conniey merged commit 4c84256 into Azure:master Oct 2, 2020
@conniey conniey deleted the feature/verify-peer branch October 2, 2020 15:54
YijunXieMS pushed a commit that referenced this pull request Oct 9, 2020
* Adding StrictTlsContext and StrictTlsContextSpi.

* Add overloads to ConnectionHandler and WebSocketsConnectionHandler to verify peer.

* Removing MockReactorHandlerProvider and fixing test breaks.

* Adding tests for StrictTlsContextSpi.

* Ensure ClientOptions is not null.

* Adding support for Service Bus and Event Hubs.

* Replacing deprecated with verifyNoInteractions.

* Fixing Proxy Send test because we do not self-sign certs

(cherry picked from commit 4c84256)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Azure.Core azure-core Client This issue points to a problem in the data-plane of the library. Event Hubs Service Bus

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants