Jakarta-Branch Rebase auf aktuellen Master

readme.md

@@ -1,8 +1,11 @@
 ## Vorab
 
-Dies ist die offizielle Quelle von HBCI4Java,
-welches u.a. in [Hibiscus](http://www.willuhn.de/products/hibiscus)
-zum Einsatz kommt.
+Dies ist die offizielle Quelle von HBCI4Java, welches u.a. in [Hibiscus](http://www.willuhn.de/products/hibiscus) zum Einsatz kommt.
+
+## Versionshinweis
+
+- Version 3.x von HBCI4Java verwendet Java EE (javax.*).
+- Version 4.x von HBCI4Java verwendet Jakarta EE (jakarta.*).
 
 ## Kontakt
 
@@ -23,7 +26,8 @@ Seither wurden umfangreiche neue Features hinzugefügt wie etwa:
 - Support für alle aktuellen SEPA-PAIN-Versionen
 - Unterstützung von PC/SC-Kartenlesern via javax.smartcardio API
 - Eine aktuelle Bankenliste (mit BLZ, Server-Adresse, HBCI-Version,...)
-- Die Unterstützung der neuen TAN-Verfahren (smsTAN, photoTAN, chipTAN - incl. Implementierung des HHD-Standards mit Flicker-Code)
+- Unterstützung der neuen TAN-Verfahren (smsTAN, photoTAN, chipTAN - incl. Implementierung des HHD-Standards mit Flicker-Code)
+- Unterstützung von RAH10-Schlüsseldateien
 
 ## Lizenz
 

src/main/java/org/kapott/hbci/GV/GVDauerSEPADel.java

@@ -75,7 +75,7 @@ public GVDauerSEPADel(HBCIHandler handler) {
         addConstraint("src.bic",   "sepa.src.bic",   null, LogFilter.FILTER_MOST);
         addConstraint("src.iban",  "sepa.src.iban",  null, LogFilter.FILTER_IDS);
         addConstraint("src.name",  "sepa.src.name",  null, LogFilter.FILTER_IDS);
-        addConstraint("dst.bic",   "sepa.dst.bic",   null, LogFilter.FILTER_MOST);
+        addConstraint("dst.bic",   "sepa.dst.bic",   "",   LogFilter.FILTER_MOST); // Kann eventuell entfallen, da BIC optional
         addConstraint("dst.iban",  "sepa.dst.iban",  null, LogFilter.FILTER_IDS);
         addConstraint("dst.name",  "sepa.dst.name",  null, LogFilter.FILTER_IDS);
         addConstraint("btg.value", "sepa.btg.value", null, LogFilter.FILTER_NONE);

src/main/java/org/kapott/hbci/GV/GVDauerSEPAEdit.java

@@ -71,7 +71,7 @@ public GVDauerSEPAEdit(HBCIHandler handler) {
         addConstraint("src.bic",   "sepa.src.bic",   null, LogFilter.FILTER_MOST);
         addConstraint("src.iban",  "sepa.src.iban",  null, LogFilter.FILTER_IDS);
         addConstraint("src.name",  "sepa.src.name",  null, LogFilter.FILTER_IDS);
-        addConstraint("dst.bic",   "sepa.dst.bic",   null, LogFilter.FILTER_MOST);
+        addConstraint("dst.bic",   "sepa.dst.bic",   "",   LogFilter.FILTER_MOST); // Kann eventuell entfallen, da BIC optional
         addConstraint("dst.iban",  "sepa.dst.iban",  null, LogFilter.FILTER_IDS);
         addConstraint("dst.name",  "sepa.dst.name",  null, LogFilter.FILTER_IDS);
         addConstraint("btg.value", "sepa.btg.value", null, LogFilter.FILTER_NONE);

src/main/java/org/kapott/hbci/GV/GVSaldoReq.java

@@ -50,17 +50,17 @@ public GVSaldoReq(HBCIHandler handler)
     {
         this(handler,getLowlevelName());
 
-
-        boolean sepa = false;
+        int version = -1;
         try
         {
-          sepa = Integer.parseInt(this.getSegVersion()) >= 7; 
+          version = Integer.parseInt(this.getSegVersion());
         }
         catch (Exception e)
         {
           HBCIUtils.log(e);
         }
-
+
+        boolean sepa = version >= 7;
         boolean nat = this.canNationalAcc(handler);
 
         if (sepa)
@@ -69,9 +69,12 @@ public GVSaldoReq(HBCIHandler handler)
           addConstraint("my.iban","KTV.iban",null, LogFilter.FILTER_IDS);
         }
 
+        // Die DE mit der Währung wurde in HKSAL5 entfernt
+        if (version < 5)
+          addConstraint("my.curr","curr","EUR", LogFilter.FILTER_NONE);
+
         if (nat || !sepa)
         {
-          addConstraint("my.curr","curr","EUR", LogFilter.FILTER_NONE);
           addConstraint("my.country","KTV.KIK.country","DE", LogFilter.FILTER_NONE);
           addConstraint("my.blz","KTV.KIK.blz",null, LogFilter.FILTER_MOST);
           addConstraint("my.number","KTV.number",null, LogFilter.FILTER_IDS);

src/main/java/org/kapott/hbci/GV/GVTAN2Step.java

@@ -25,10 +25,14 @@
 import java.util.Properties;
 
 import org.kapott.hbci.GV_Result.GVRSaldoReq;
+import org.kapott.hbci.callback.HBCICallback;
 import org.kapott.hbci.dialog.KnownReturncode;
 import org.kapott.hbci.dialog.KnownTANProcess;
+import org.kapott.hbci.exceptions.HBCI_Exception;
+import org.kapott.hbci.manager.Feature;
 import org.kapott.hbci.manager.HBCIHandler;
 import org.kapott.hbci.manager.HBCIUtils;
+import org.kapott.hbci.manager.HBCIUtilsInternal;
 import org.kapott.hbci.manager.LogFilter;
 import org.kapott.hbci.passport.AbstractPinTanPassport;
 import org.kapott.hbci.passport.HBCIPassportInternal;
@@ -49,6 +53,9 @@ public class GVTAN2Step extends HBCIJobImpl
     private HBCIJobImpl task;
 
     private HBCIJobImpl redo;
+
+    // Die bisherige Anzahl von Decoupled Status refresh requests.
+    private int decoupledRefreshes = 0;
 
     public static String getLowlevelName()
     {
@@ -177,6 +184,15 @@ protected void saveReturnValues(HBCIMsgStatus status, int sref)
     @Override
     public HBCIJobImpl redo()
     {
+        // Falls der redo job 'this' ist, wird ein redo für das Decoupled Verfahren durchgeführt, welcher jeweils
+        // nur ein mal pro 3956 status wiederholt werden soll. Um unendliche Wiederholungen zu vermeiden setzen wir
+        // also redo=null.
+        if (this.redo == this)
+        {
+            HBCIJobImpl redo = this.redo;
+            this.redo = null;
+            return redo;
+        }
         return this.redo;
     }
 
@@ -203,16 +219,52 @@ protected void extractResults(HBCIMsgStatus msgstatus,String header,int idx)
         ///////////////////////////////////////////////////////////////////////
         // Die folgenden Sonderbehandlungen sind nur bei Prozess-Variante 2 in Schritt 2 noetig,
         // weil wir dort ein Response auf einen GV erhalten, wir selbst aber gar nicht der GV sind sondern das HKTAN Step2
-        if (this.process == KnownTANProcess.PROCESS2_STEP2 && this.task != null)
+        if ((this.process == KnownTANProcess.PROCESS2_STEP2 || this.process == KnownTANProcess.PROCESS2_STEPS) && this.task != null)
         {
-            // Pruefen, ob die Bank eventuell ein 3040 gesendet hat - sie also noch weitere Daten braucht.
-            // Das 3040 bezieht sich dann aber nicht auf unser HKTAN sondern auf den eigentlichen GV
-            // In dem Fall muessen wir dem eigentlichen Task mitteilen, dass er erneut ausgefuehrt werden soll.
-            if (StringUtil.toInsCode(this.getHBCICode()).equals(segCode) && KnownReturncode.W3040.searchReturnValue(msgstatus.segStatus.getWarnings()) != null && this.task.redoAllowed())
-            {
-                HBCIUtils.log("found status code 3040, need to repeat task " + this.task.getHBCICode(),HBCIUtils.LOG_DEBUG);
-                HBCIUtils.log("Weitere Daten folgen",HBCIUtils.LOG_INFO);
-                this.redo = this.task;
+            if (StringUtil.toInsCode(this.getHBCICode()).equals(segCode)) {
+                if (KnownReturncode.W3040.searchReturnValue(msgstatus.segStatus.getWarnings()) != null && this.task.redoAllowed()) {
+                    // Pruefen, ob die Bank eventuell ein 3040 gesendet hat - sie also noch weitere Daten braucht.
+                    // Das 3040 bezieht sich dann aber nicht auf unser HKTAN sondern auf den eigentlichen GV
+                    // In dem Fall muessen wir dem eigentlichen Task mitteilen, dass er erneut ausgefuehrt werden soll.
+                    HBCIUtils.log("found status code 3040, need to repeat task " + this.task.getHBCICode(),HBCIUtils.LOG_DEBUG);
+                    HBCIUtils.log("Weitere Daten folgen",HBCIUtils.LOG_INFO);
+                    this.redo = this.task;
+                } else if (Feature.PINTAN_DECOUPLED_REFRESH.isEnabled() && KnownReturncode.W3956.searchReturnValue(msgstatus.segStatus.getWarnings()) != null) {
+                    // Beim Decoupled Verfahren kann die Bank ein 3956 senden, wenn der Nutzer den Prozess noch nicht bestätigt hat.
+                    // In diesem Fall muss dieser task wiederholt werden, um erneut zu prüfen, ob die Bestätigung erfolgt ist.
+                    // Wir benachrichtigen die Applikation mit einem entsprechenden Callback und warten eine mögliche Mindestzeit.
+                    HBCIUtils.log("found status code 3956, need to repeat task " + this.getHBCICode(),HBCIUtils.LOG_DEBUG);
+                    AbstractPinTanPassport pinTanPassport = (AbstractPinTanPassport) getMainPassport();
+                    if (pinTanPassport.getDecoupledMaxRefreshes() != null && this.decoupledRefreshes >= pinTanPassport.getDecoupledMaxRefreshes()) {
+                        throw new HBCI_Exception("*** the maximum number of decoupled refreshes has been reached.");
+                    }
+                    Integer timeBeforeDecoupledRefresh = this.decoupledRefreshes == 0
+                            ? pinTanPassport.getMinimumTimeBeforeFirstDecoupledRefresh()
+                            : pinTanPassport.getMinimumTimeBeforeNextDecoupledRefresh();
+                    long callbackDurationMs = System.currentTimeMillis();
+                    HBCIUtilsInternal.getCallback().callback(
+                            getMainPassport(),
+                            HBCICallback.NEED_PT_DECOUPLED_RETRY,
+                            "*** decoupled SCA still required",
+                            HBCICallback.TYPE_TEXT,
+                            new StringBuffer(String.valueOf(timeBeforeDecoupledRefresh != null ? timeBeforeDecoupledRefresh : 0)));
+                    callbackDurationMs = System.currentTimeMillis() - callbackDurationMs;
+                    if (timeBeforeDecoupledRefresh != null && callbackDurationMs < timeBeforeDecoupledRefresh * 1000) {
+                        long sleepMs = timeBeforeDecoupledRefresh * 1000 - callbackDurationMs;
+                        HBCIUtils.log(String.format(
+                                "The pause before the next decoupled request was too short. Sleeping for %dms to reach the required delay.", sleepMs
+                        ),HBCIUtils.LOG_INFO);
+                        try {
+                            Thread.sleep(sleepMs);
+                        } catch (InterruptedException e) {
+                            throw new HBCI_Exception("*** Decoupled refresh sleep was interrupted.");
+                        }
+                    }
+                    this.decoupledRefreshes++;
+                    this.redo = this;
+                } else {
+                    this.redo = null;
+                }
             }
 
             // Das ist das Response auf den eigentlichen GV - an den Task durchreichen
@@ -222,7 +274,7 @@ protected void extractResults(HBCIMsgStatus msgstatus,String header,int idx)
                 HBCIUtils.log("this is a response segment for the original task (" + this.task.getName() + ") - storing results in the original job",HBCIUtils.LOG_DEBUG);
                 this.task.fillJobResultFromTanJob(msgstatus, header, idx);
             }
-            
+
             // Wir haben hier nichts weiter zu tun
             return;
         }
@@ -252,7 +304,6 @@ protected void extractResults(HBCIMsgStatus msgstatus,String header,int idx)
         ///////////////////////////////////////////////////////////////////////
         // Daten fuer die TAN-Abfrage einsammeln
 
-        // Prozess-Variante 1:
         final String challenge = result.getProperty(header+".challenge");
         if (challenge != null)
         {

src/main/java/org/kapott/hbci/GV/generators/GenLastSEPA00800108.java

@@ -158,8 +158,10 @@ private DirectDebitTransactionInformation23 createDirectDebitTransactionInformat
 
     final GenericPersonIdentification1 creditor = new GenericPersonIdentification1();
     creditor.setId(sepaParams.getProperty(SepaUtil.insertIndex("creditorid", index)));
+
     final PersonIdentificationSchemeName1Choice schema = new PersonIdentificationSchemeName1Choice();
     schema.setPrtry("SEPA");
+    creditor.setSchmeNm(schema);
     drctDbtTxInf.getDrctDbtTx().getCdtrSchmeId().getId().getPrvtId().getOthr().add(creditor);
 
     drctDbtTxInf.getDrctDbtTx().setMndtRltdInf(new MandateRelatedInformation14());
@@ -232,7 +234,12 @@ private DirectDebitTransactionInformation23 createDirectDebitTransactionInformat
     drctDbtTxInf.setInstdAmt(new ActiveOrHistoricCurrencyAndAmount());
     drctDbtTxInf.getInstdAmt().setValue(new BigDecimal(sepaParams.getProperty(SepaUtil.insertIndex("btg.value", index))));
 
-    drctDbtTxInf.getInstdAmt().setCcy("EUR");
+    String currencyCode = sepaParams.getProperty(SepaUtil.insertIndex("btg.curr", index));
+    if(currencyCode != null && currencyCode.length() > 0) {
+      drctDbtTxInf.getInstdAmt().setCcy(currencyCode);
+    } else {
+      drctDbtTxInf.getInstdAmt().setCcy("EUR");
+    }
 
     // Payment Information - Credit Transfer Transaction Information - Usage
     String usage = sepaParams.getProperty(SepaUtil.insertIndex("usage", index));

src/main/java/org/kapott/hbci/GV/generators/GenLastSEPA00800110.java

@@ -160,6 +160,7 @@ private DirectDebitTransactionInformation28 createDirectDebitTransactionInformat
     creditor.setId(sepaParams.getProperty(SepaUtil.insertIndex("creditorid", index)));
     final PersonIdentificationSchemeName1Choice schema = new PersonIdentificationSchemeName1Choice();
     schema.setPrtry("SEPA");
+    creditor.setSchmeNm(schema);
     drctDbtTxInf.getDrctDbtTx().getCdtrSchmeId().getId().getPrvtId().getOthr().add(creditor);
 
     drctDbtTxInf.getDrctDbtTx().setMndtRltdInf(new MandateRelatedInformation15());
@@ -232,7 +233,12 @@ private DirectDebitTransactionInformation28 createDirectDebitTransactionInformat
     drctDbtTxInf.setInstdAmt(new ActiveOrHistoricCurrencyAndAmount());
     drctDbtTxInf.getInstdAmt().setValue(new BigDecimal(sepaParams.getProperty(SepaUtil.insertIndex("btg.value", index))));
 
-    drctDbtTxInf.getInstdAmt().setCcy("EUR");
+    String currencyCode = sepaParams.getProperty(SepaUtil.insertIndex("btg.curr", index));
+    if(currencyCode != null && currencyCode.length() > 0) {
+      drctDbtTxInf.getInstdAmt().setCcy(currencyCode);
+    } else {
+      drctDbtTxInf.getInstdAmt().setCcy("EUR");
+    }
 
     // Payment Information - Credit Transfer Transaction Information - Usage
     String usage = sepaParams.getProperty(SepaUtil.insertIndex("usage", index));

src/main/java/org/kapott/hbci/GV/generators/GenUebSEPA00100102.java

@@ -163,7 +163,12 @@ private CreditTransferTransactionInformation2 createCreditTransferTransactionInf
         cdtTrxTxInf.getAmt().setInstdAmt(new EuroMax9Amount());
         cdtTrxTxInf.getAmt().getInstdAmt().setValue(new BigDecimal(sepaParams.getProperty(SepaUtil.insertIndex("btg.value", index))));
 
-        cdtTrxTxInf.getAmt().getInstdAmt().setCcy("EUR");
+        String currencyCode = sepaParams.getProperty(SepaUtil.insertIndex("btg.curr", index));
+        if(currencyCode != null && currencyCode.length() > 0) {
+          cdtTrxTxInf.getAmt().getInstdAmt().setCcy(currencyCode);
+        } else {
+          cdtTrxTxInf.getAmt().getInstdAmt().setCcy("EUR");
+        }
 
         //Payment Information - Credit Transfer Transaction Information - Usage
         String usage = sepaParams.getProperty(SepaUtil.insertIndex("usage", index));

src/main/java/org/kapott/hbci/GV/generators/GenUebSEPA00100109.java

@@ -186,7 +186,12 @@ private CreditTransferTransaction34 createCreditTransferTransaction(Properties s
         cdtTrxTxInf.getAmt().setInstdAmt(new ActiveOrHistoricCurrencyAndAmount());
         cdtTrxTxInf.getAmt().getInstdAmt().setValue(new BigDecimal(sepaParams.getProperty(SepaUtil.insertIndex("btg.value", index))));
 
-        cdtTrxTxInf.getAmt().getInstdAmt().setCcy("EUR");
+        String currencyCode = sepaParams.getProperty(SepaUtil.insertIndex("btg.curr", index));
+        if(currencyCode != null && currencyCode.length() > 0) {
+          cdtTrxTxInf.getAmt().getInstdAmt().setCcy(currencyCode);
+        } else {
+          cdtTrxTxInf.getAmt().getInstdAmt().setCcy("EUR");
+        }
 
         //Payment Information - Credit Transfer Transaction Information - Usage
         String usage = sepaParams.getProperty(SepaUtil.insertIndex("usage", index));

src/main/java/org/kapott/hbci/GV/generators/GenUebSEPA00100111.java

@@ -186,7 +186,12 @@ private CreditTransferTransaction54 createCreditTransferTransaction(Properties s
         cdtTrxTxInf.getAmt().setInstdAmt(new ActiveOrHistoricCurrencyAndAmount());
         cdtTrxTxInf.getAmt().getInstdAmt().setValue(new BigDecimal(sepaParams.getProperty(SepaUtil.insertIndex("btg.value", index))));
 
-        cdtTrxTxInf.getAmt().getInstdAmt().setCcy("EUR");
+        String currencyCode = sepaParams.getProperty(SepaUtil.insertIndex("btg.curr", index));
+        if(currencyCode != null && currencyCode.length() > 0) {
+          cdtTrxTxInf.getAmt().getInstdAmt().setCcy(currencyCode);
+        } else {
+          cdtTrxTxInf.getAmt().getInstdAmt().setCcy("EUR");
+        }
 
         //Payment Information - Credit Transfer Transaction Information - Usage
         String usage = sepaParams.getProperty(SepaUtil.insertIndex("usage", index));