Skip to content

TLS reports #55

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

Open
wants to merge 36 commits into
base: master
Choose a base branch
from
Open

TLS reports #55

wants to merge 36 commits into from

Conversation

kimboslice99
Copy link

$message isnt defined in the event there isnt new reports and $reports is not empty.

$folder parameter wasnt being used

Folder was missing the trailing /

X-Xss-Protection should be disabled

RvdHout and others added 30 commits May 2, 2021 23:06
check emailaddress forwarding account
Oops forgot the other file
A few more conditional checks, there a probably plenty more but this is a good start
Additions to make it compatible with 5.6.8-B2574.40
5.6.9-B2602.53
Log failed logins
As pointed out by kimboslice99
962f46f
5.7.0 compatibility
Oops, fixed regex
this could probably be improved
This reverts commit 0aa85d8.
still some weird nesting issue happening, hmm
now to take care of failure details
failure report handling, needs more work, i think
more to be done
only show failure details header once.  maybe add a break between each table
if the naming of the report follows the recommendation in RFC 8460 - 5.1 this should be fine
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants