-
Notifications
You must be signed in to change notification settings - Fork 56
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
Feature: TLS-RPT Support in the Viewer #73
Comments
I've made a pull request to add TLS-RPT functionality in the v2.0 branch of the viewer. I've also made a pull request to the parser that populates the database with the TLS-RPT data. This code should be considered alpha quality not suitable for use in a production environment. I have done some basic testing and everything seems to work except the following:
For some reason, I can't get SMTP TLS Reporting to work with my domains so all I get are failure reports. As a result, I don't really know what a successful TLS-RPT looks like. If someone wants to post some here so I can test them, it would be appreciated. @ekalin, could you please review my PostgreSQL code if you are able. I have tried to duplicate your PostgreSQL additions as closely as possible but I don't have a PostgreSQL server on which to test. |
Here's the contents of a valid report:
I'll try to get an actual email message with the attached report. |
Thanks for the report. So I guess I did get TLS-RPT to work because that's the same type of report that I get, although I get failure details as well. I'll work on getting the icons to show up properly. |
And here's a file with a report, in case it helps. And now I need a report with failures, to test the queries against the second table. |
Thanks!
Ooh, I have lots of those :-) |
Insertion and retrieval from the Postres db worked fine, so it seems the database issues are resolved. |
Thanks for reviewing that code! |
Starting to test the viewer changes today and so far, so good. |
I've been working on an enhancement to dmarcts-report-parser to add TLS reporting (see Issue 65). Since any addition to the parser would be useless without a corresponding addition to the viewer, I have started modifying the viewer to incorporate TLS reports.
As a proof-of-concept, I simply copied the dmarcts*.php files to tlsts*.php files and quickly modified them to display the TLS reports. I have managed to get most of the features working in TLS reports (JSON highlighting is eluding me at the moment).
What I have found is that the main files (dmarcts- and tlsts-report-viewer.php) and the *-list.php files are similar enough that I can merge them and deal with the differences in one set of files (say, viewer.php and viewer-list.php). I haven't yet made a decision about the *-data files; they are different enough to maybe warrant separate files (say, tls-viewer-data.php and dmarc-viewer-data.php). I will have to do some more investigation.
Because this feature introduces major new functionality, I think it warrants a version bump. @techsneeze, could you please create a v2.0 branch or something similar? Also, maybe label this issue "enhancement"?
The text was updated successfully, but these errors were encountered: