Overriding Net::SSH::Config.default_files is not the best strategy as of today
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 35/100
Hướng nghiên cứu
Bắt đầu bằng cách đọc lib/sshkit/backends/netssh.rb tại dòng được liên kết và so sánh cách SSHKit ghi đè Net::SSH.default_files bằng tùy chọn :config được mô tả trong issue. Xem lại net-ssh issue #587 để hiểu hành vi của ProxyJump. Được xem là hoàn tất khi xác định được SSHKit nên sử dụng đường dẫn cấu hình nào và xác nhận rằng ProxyJump hoạt động mà không gặp sự không khớp cấu hình hiện tại.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Hi, I'm currently unable to use the ProxyJump config with sshkit, due to this issue:
https://github.com/net-ssh/net-ssh/issues/587
sshkit will override Net::SSH.default_files here:
By the way, is there any reasons to do that instead of something simpler like:
Net::SSH::Config.default_files << File.join(Dir.pwd, '.ssh/config')
Either way, net-ssh doesn't work the same way when the :config option is used in Net::SSH.start and when passed to default_files, as you could see in the issue I mentioned above.
Currently I'm in a rush to try to meet some tight deadlines for a new project I started and it will last a few months at least. If I were to find some time to contribute it would be more likely to provide some patch to sshkit to use the :config option instead of overriding default_files as it seems a simpler/quicker change for someone new to both projects, in my opinion. Before I do so, I'd like to check on your opinion. Is there anyone here that is comfortable with net-ssh source and would be willing to contribute to it by providing the "-F" option to the ssh client for each of those default_files? Otherwise, would you be willing to accept a patch that would change the current approach in sshkit so that this default configuration would be passed to the :config option instead of overriding default_files?
Thanks for your time to read all of this :)
- Ngôn ngữ chính
- Ruby
- Star
- 1.2k
- Fork
- 257
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
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.
Issue khác của capistrano/sshkit
-
should have an openssh backend Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
capistrano/sshkit#563 · 6 reaction ·
-
No redaction for with values Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 45/100
capistrano/sshkit#562 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
capistrano/sshkit#554 · 3 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
capistrano/sshkit#543 · 2 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 45/100
capistrano/sshkit#518 · 1 bình luận ·
Tất cả issue của capistrano/sshkit
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Please Drop ruby 3.2 support Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
googleapis/google-api-ruby-client#28001 · 5 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100