microsoft / microsoft/libHttpClient
There should be a version of HCHttpCallRequestSetSSLValidation for web sockets
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- C++
- Star
- 295
- Fork
- 144
- Merge trung bình
- 2 ngày 19 giờ
- Pull request đã merge (30 ngày)
- 2
Mô tả
I am adding support for Web Sockets to our title, and have been testing my work with a very simple locally run echo server. Everything works fine when testing over http, but switching to https fails as I am just using a self signed certificate for local testing.
For making HTTP calls, I call HCHttpCallRequestSetSSLValidation to skip the ssl validation, but there is no equivilent for web sockets. I found HCWebSocketSetProxyDecryptsHttps, and using a debugger to skip the empty proxy check was able to get a connection open over https with the ssl check bypassed. This is not sustainable in the long term, as I can't expect other developers to stick a breakpoint in a random bit of code and set a random variable to true.
Please consider adding a new function along the lines of HCWebSocketSetSSLValidation, which when called disables the SSL validation step.
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu bằng cách đọc các entry point hiện có HCHttpCallRequestSetSSLValidation và HCWebSocketSetProxyDecryptsHttps để hiểu cách kiểm soát việc xác thực SSL cho các kết nối HTTP và WebSocket. Thêm API tương ứng dành cho WebSocket được mô tả trong issue, sau đó xác minh rằng một máy chủ echo WebSocket cục bộ sử dụng chứng chỉ tự ký có thể kết nối khi việc xác thực bị tắt.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- cpp
- Lĩnh vực
- api, security
- Loại issue
- Tính năng
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Ít trao đổi
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 54/100