fix proxy.md

[jiangying000]

related: #821

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.34%. Comparing base (ba715b9) to head (05d222b).
Report is 66 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #837      +/-   ##
==========================================
- Coverage   71.95%   70.34%   -1.61%     
==========================================
  Files          91      113      +22     
  Lines        8001     8896     +895     
==========================================
+ Hits         5757     6258     +501     
- Misses       2244     2638     +394     

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

[dblock]

Why doesn't it work with the default connection class?

[jiangying000]

Why doesn't it work with the default connection class?

i don't know, but it does not work before I added connection_class=RequestsHttpConnection,

[dblock]

RequestsHttpConnection

There are multiple connection classes, see https://github.com/opensearch-project/opensearch-py/blob/9dd7bf7cb3e80b98fcc256edb9c49cf89202f522/guides/connection_classes.md.

The default is Urllib3HttpConnection for synchronous scenarios. The change you are proposing switching a non-working proxy implementation in Urllib3HttpConnection to a working proxy implementation in RequestsHttpConnection, so I don't think this is the right fix, aside of the documentation someone needs to actually fix the problem for proxy support in Urllib3HttpConnection.

[jiangying000]

Agree, this is not a fix to #821 , just a temporary fix to the doc

At least Users can get what they want by following the doc after this change

[dblock]

I agree, I'll merge this.