HTTPS proxy is only enabled if the proxied connection is also SSL
Nessuno ha ancora preso questa issue.
- Lingua principale
- Ruby
- Stelle
- 148
- Fork
- 95
- Merge medio
- 10h 54m
- PR unite (30g)
- 4
Descrizione
irb(main):001> require 'net/http'
=> true
irb(main):002> n = Net::HTTP.new('192.0.2.10', 80, '127.0.0.1', 5000, nil, nil, nil, true)
=> #<Net::HTTP 192.0.2.10:80 open=false>
irb(main):003> n.instance_variable_get(:@proxy_use_ssl)
=> true
irb(main):004> n.start
=> #<Net::HTTP 192.0.2.10:80 open=true>
irb(main):005> n.get('/')
Data sent to the listener at 127.0.0.1:5000:
GET http://192.0.2.10/ HTTP/1.1
Accept-Encoding: gzip;q=1.0,deflate;q=0.6,identity;q=0.3
Accept: */*
User-Agent: Ruby
Host: 192.0.2.10
This is due to the proxy SSL socket setup being inside a if use_ssl? block.
https://github.com/ruby/net-http/blob/master/lib/net/http.rb#L1667-L1674
Transporting clear HTTP over a HTTPS proxy is a valid configuration, the library should support this.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia in lib/net/http.rb intorno alle righe 1667-1674 e riproduci la configurazione Net::HTTP segnalata, con una destinazione HTTP chiara e un proxy HTTPS. Traccia la configurazione del socket del proxy e verifica che le richieste HTTP in chiaro possano essere trasportate tramite un proxy HTTPS senza modificare la connessione alla destinazione. Conferma il comportamento con un test di regressione mirato usando la configurazione mostrata.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- ruby
- Ambito
- networking
- Tipo di issue
- Bug
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 48/100