ngx_mruby icon indicating copy to clipboard operation
ngx_mruby copied to clipboard

Add ssl verify client handler

Open keizo042 opened this issue 7 years ago • 4 comments

Pull-Request Check List

The patch adds handlers after verifying client certificate in SSL Handshake.

keizo042 avatar Feb 03 '18 17:02 keizo042

I would fix things you point out. Please review them.

keizo042 avatar Feb 06 '18 13:02 keizo042

@keizo042 この件いかがでしょう?とても良いPRなのでマージしたいです

matsumotory avatar Feb 14 '18 03:02 matsumotory

Nginx::SSL.accept_clientのテストを書くのが手間で止まっています。 他の指摘は問題なく修正可能です。

Nginx:SSL.accept_clientのテストには x509として有効であり値として不適切である自己署名ではないx509証明書が必要です。 これを作成する方法とそれが適切であるとする条件が纏めきれていません。

証明書発行まわりは不勉強なので時間がかかりそうです。

keizo042 avatar Feb 14 '18 08:02 keizo042

@matsumotory I have tried to use this feature for my local project and I start to confuse. Because, I have no idea what will be use case for this feature. As far I know MRuby doesn't have openssl library like CRuby. So, we cannot validate client certificate based with our custom CA cert.

Also, about a test I think openssl s_client not working properly for client cert validation results. It was only giving error "1: self signed certificate" when you provide a wrong client certificate, also same thing happened if you did not provide -CAfile abc.cert in command.

Zogoo avatar Jan 21 '20 06:01 Zogoo