search

ThreeMammals / Ocelot

.NET API Gateway
https://www.nuget.org/packages/Ocelot
MIT License
8.31k stars 1.63k forks source link

The SSL connection could not be established - While setting up LoadBalancer using Consul and LeastConnection. #1811

Closed raman-m closed 2 months ago

raman-m commented 9 months ago

Discussed in https://github.com/ThreeMammals/Ocelot/discussions/1804

Originally posted by **ks1990cn** November 24, 2023 Hi @raman-m @ggnaegi / Anyone , Can you please help me to resolve error mentioned in title? I've created 3 web api projects - 2 of them to replicate DownstreamApi Instance1 and Instance2, 1 is to replicate API Gateway. when I try to reach downstream using upstream of API Gateway, and when it tries to find hostandport anc check for services available, it suddenly breaks stating SSL Connection could not be established. Can you help me by cloning below code and let me know what I am missing in setup. Here is GIF of problem and code from where you can clone. Please let me know any input required from me. ![SSLissue](https://github.com/ThreeMammals/Ocelot/assets/29522704/8d8b9bc8-840a-4806-851f-29695ed6b572) Code path changes I've done: https://github.com/ks1990cn/Ocelot/tree/LoadBalancerSample
raman-m commented 9 months ago

Hello @ks1990cn! Good you answer here plz? This is your official ticket. 😉

I'll update the description later...

ks1990cn commented 9 months ago

Thank you so much @raman-m .

Actually I am done with question, just I want code review to add sample code demonstrating LoadBalancer with Consul.

That I am using alot while working on backlog items to replicate. I can also add on quick documentation so anyone can quickly change on that configuration locally and replicate any bug.

Just to do manual testing also we can have config set up.

raman-m commented 9 months ago

You are assigned officially! 😉 I'll review the PR later today... Good luck!

raman-m commented 9 months ago

@ks1990cn commented on Nov 29:

Actually I am done with question, just I want code review to add sample code demonstrating LoadBalancer with Consul.

Is this a general idea and goal/task of this issue?

ks1990cn commented 9 months ago

@raman-m commented on Dec 2

I feel its a general idea, cause I am able to replicate most of the issues using this sample configuration. And its for helping anyone going ahead to easy start with debugging and replicating any issue (If any new person wants to start contribution)

raman-m commented 2 months ago

You have the right to reopen this issue in the future if you come with a PR ready.