-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Feat: Improve TLS certificate display & handling #4281
base: master
Are you sure you want to change the base?
Conversation
More as a discussion point: Should this be backported or are there any breaking changes that I am not seeing |
The |
I would prefer if you were to split this PR into the 1.23.X part and the 2.x part. |
Co-authored-by: Adam Stachowicz <saibamenppl@gmail.com>
https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md#can-i-create-a-pull-request-for-uptime-kuma
Tick the checkbox if you understand [x]:
Description
Fixes #3551
keylog
event should be more reliable.keylog
event, whether the connection is valid and secure is not defined. Ideally we would listen for thesecureConnect
event for this, but usingaxios
there does not seem to be a way to do this. Hence it is manually checked in the previous location.authorizationError
information and display it in the UI.Type of change
Checklist
Screenshots (if any)