[ https://issues.apache.org/jira/browse/NIFI-1899?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15368267#comment-15368267 ]
ASF GitHub Bot commented on NIFI-1899: -------------------------------------- Github user JPercivall commented on a diff in the pull request: https://github.com/apache/nifi/pull/483#discussion_r70128375 --- Diff: nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/event/SmtpEvent.java --- @@ -0,0 +1,91 @@ +/* +* Licensed to the Apache Software Foundation (ASF) under one or more +* contributor license agreements. See the NOTICE file distributed with +* this work for additional information regarding copyright ownership. +* The ASF licenses this file to You under the Apache License, Version 2.0 +* (the "License"); you may not use this file except in compliance with +* the License. You may obtain a copy of the License at +* +* http://www.apache.org/licenses/LICENSE-2.0 +* +* Unless required by applicable law or agreed to in writing, software +* distributed under the License is distributed on an "AS IS" BASIS, +* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +* See the License for the specific language governing permissions and +* limitations under the License. +*/ + +package org.apache.nifi.processors.email.smtp.event; + + +import java.security.cert.X509Certificate; +import java.util.ArrayList; +import java.util.HashMap; +import java.util.List; +import java.util.Map; + +/** + * A Smtp event which adds the transaction number and command to the StandardEvent. + */ + +public class SmtpEvent{ + + private final String remoteIP; + private final String helo; + private final String from; + private final String to; + private final byte[] bodyData; + private List<Map<String, String>> certificatesDetails; + + public SmtpEvent(final String remoteIP, final String helo, final String from, final String to, final X509Certificate[] certificates, final byte[] bodyData) { + this.remoteIP = remoteIP; + this.helo = helo; + this.from = from; + this.to = to; + this.bodyData = bodyData; + + this.certificatesDetails = new ArrayList<>(); + + for (int c = 0; c < certificates.length; c++) { + X509Certificate cert = certificates[c]; + if (cert.getSerialNumber() != null && cert.getSubjectDN() != null) { + Map<String, String> certificate = new HashMap<>(); + + String certSerialNumber = cert.getSerialNumber().toString(); + String certSubjectDN = cert.getSubjectDN().getName(); + + + certificate.put("SerialNumber", certSerialNumber); + certificate.put("SubjectName", certSubjectDN); + + certificatesDetails.add(certificate); + + } + } + } + + public List getCertifcateDetails() { --- End diff -- Bit of a nit pick but, it is best practice to keep the explicit type here (List<Map<String, String>>) that way you don't have the "unchecked assignment" warnings in ListenSMTP. > Create ListenSMTP & ExtractEmailAttachment processors > ----------------------------------------------------- > > Key: NIFI-1899 > URL: https://issues.apache.org/jira/browse/NIFI-1899 > Project: Apache NiFi > Issue Type: New Feature > Reporter: Andre > -- This message was sent by Atlassian JIRA (v6.3.4#6332)