-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add SubjectAltName into certificate #28
base: master
Are you sure you want to change the base?
Changes from all commits
26a722d
70a64e3
883accb
69878a8
729310d
e790605
820eddf
1743ebf
e9afa0f
f480b06
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -34,7 +34,13 @@ def prepare_sni_callback(ctx, ca) | |
mutex.synchronize do | ||
unless contexts.include? servername | ||
cert = Ritm::Certificate.create(servername) | ||
ca.sign(cert) | ||
extensions = Ritm::CA.signing_profile | ||
extensions['extensions']['subjectAltName'] = { | ||
'dns_names' => [servername], | ||
'uris' => [servername] | ||
} | ||
Comment on lines
+38
to
+41
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix "Subject Alternative Name Missing" in browser |
||
ca.sign(cert, extensions) | ||
|
||
contexts[servername] = context_with_cert(sock.context, cert) | ||
end | ||
end | ||
|
@@ -64,3 +70,4 @@ def duplicate_context(original_ctx) | |
end | ||
end | ||
end | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -44,6 +44,7 @@ def build_settings(session) | |
def build_proxy | ||
@http = Ritm::Proxy::ProxyServer.new(BindAddress: @conf.proxy.bind_address, | ||
Port: @conf.proxy.bind_port, | ||
ProxyAuthProc: @conf.proxy.auth_proc, | ||
AccessLog: [], | ||
Logger: WEBrick::Log.new(File.open(File::NULL, 'w')), | ||
https_forward: @https_forward, | ||
|
@@ -53,7 +54,8 @@ def build_proxy | |
end | ||
|
||
def build_reverse_proxy | ||
@https = Ritm::Proxy::SSLReverseProxy.new(@conf.ssl_reverse_proxy.bind_port, | ||
@https = Ritm::Proxy::SSLReverseProxy.new(@conf.ssl_reverse_proxy.bind_address, | ||
@conf.ssl_reverse_proxy.bind_port, | ||
Comment on lines
+57
to
+58
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Allow access from the device in the same network, this is useful when you need to inception HTTPS between an iOS app and a remote server. |
||
@certificate, | ||
@forwarder) | ||
end | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix ERR_CERT_WEAK_KEY error from browser