Skip to content
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.

Sign up for GitHub

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

Jump to bottom

"DangerousAcceptAnyServerCertificateValidator=true" when "DownstreamScheme=wss" does not work #1375

Closed
zqlovejyc opened this issue Nov 18, 2020 · 1 comment · Fixed by #1377
Closed

"DangerousAcceptAnyServerCertificateValidator=true" when "DownstreamScheme=wss" does not work #1375

zqlovejyc opened this issue Nov 18, 2020 · 1 comment · Fixed by #1377
Assignees
@zqlovejyc
Labels
bug Identified as a potential bug merged Issue has been merged to dev and is waiting for the next release

Comments

@zqlovejyc
Copy link
Contributor

zqlovejyc commented Nov 18, 2020 •
edited by raman-m
Loading 

{
      "ServiceName": "MessageCenter", 
      "UseServiceDiscovery": true,
      "UpstreamPathTemplate": "/Hub/{catchAll}",
      "UpstreamHttpMethod": [ "GET", "POST", "DELETE", "PUT" ],
      "DownstreamScheme": "wss",
      "DownstreamPathTemplate": "/{catchAll}",
      "LoadBalancerOptions": {
        "Type": "LeastConnection"
      },
      "DangerousAcceptAnyServerCertificateValidator": true
    }
@zqlovejyc zqlovejyc changed the title "DangerousAcceptAnyServerCertificateValidator=true " when "DownstreamScheme=wss" does not work "DangerousAcceptAnyServerCertificateValidator=true" when "DownstreamScheme=wss" does not work Nov 18, 2020
@raman-m raman-m mentioned this issue Jul 15, 2023
Merged
@raman-m raman-m added bug Identified as a potential bug accepted Bug or feature would be accepted as a PR or is being worked on labels Aug 21, 2023
@raman-m
Copy link
Member

raman-m commented Aug 21, 2023

The issue has been accepted due to ready PR #1377 !

raman-m added a commit that referenced this issue Sep 28, 2023
@zqlovejyc @raman-m
#1375 #1237 #925 #920 Fix DownstreamRoute DangerousAcceptAnyServerCer…
fa179bf 
…tificateValidator (#1377)

* Fix/1375 fix DownstreamRoute DangerousAcceptAnyServerCertificateValidator does not work

* Fix the exception when RoundRobin services is empty

* Fix build errors

* IDE0063 'using' statement can be simplified

* IDE0051 Private member 'StreamCopyBufferSize' is unused

* Use nameof() in string interpolations

* @RaynaldM code review

* Code review. Refactor method

* Organize folders for WebSockets feature

* Add unit tests class for WebSockets feature

* Refactor middleware to make it suitable for unit testing

* Add unit test

* Review current acceptance tests for WebSockets

* Review

---------

Co-authored-by: raman-m <[email protected]>
@raman-m raman-m added merged Issue has been merged to dev and is waiting for the next release and removed accepted Bug or feature would be accepted as a PR or is being worked on labels Sep 28, 2023
@raman-m raman-m mentioned this issue Oct 3, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zqlovejyc zqlovejyc

Labels
bug Identified as a potential bug merged Issue has been merged to dev and is waiting for the next release
Projects
None yet
Milestone
No milestone
2 participants
@raman-m @zqlovejyc