[ 
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)

Reply via email to