Changeset 6a71c23 for apps/addressbook


Ignore:
Timestamp:
Apr 26, 2016 1:50:34 PM (4 years ago)
Author:
zzz <zzz@…>
Branches:
master
Children:
f35eaaf
Parents:
fc8b7276
Message:

Addressbook: Check inner sig even if old name not found
More logging cleanup

File:
1 edited

Legend:

Unmodified
Added
Removed
  • apps/addressbook/java/src/net/i2p/addressbook/Daemon.java

    rfc8b7276 r6a71c23  
    213213                                        if (pod2 == null) {
    214214                                            // we didn't know it before, so we'll add it
    215                                             // TODO check inner sig anyway?
     215                                            // check inner sig anyway
     216                                            if (!he.hasValidInnerSig()) {
     217                                                logInner(log, action, key, addressbook);
     218                                                invalid++;
     219                                                continue;
     220                                            }
    216221                                        } else if (pod2.contains(dest)) {
    217222                                            // we knew it before, with the same dest
     
    221226                                            // checks out, so verify the inner sig
    222227                                            if (!he.hasValidInnerSig()) {
    223                                                 if (log != null)
    224                                                     log.append("Action: " + action + " failed because" +
    225                                                                " inner signature for key " + key +
    226                                                                " failed" +
    227                                                                ". From: " + addressbook.getLocation());
     228                                                logInner(log, action, key, addressbook);
    228229                                                invalid++;
    229230                                                continue;
     
    259260                                        }
    260261                                    } else {
    261                                         if (log != null)
    262                                             log.append("Action: " + action + " failed, missing required parameters");
     262                                        logMissing(log, action, key, addressbook);
    263263                                        invalid++;
    264264                                        continue;
     
    285285                                        }
    286286                                    } else {
    287                                         if (log != null)
    288                                             log.append("Action: " + action + " failed, missing required parameters" +
    289                                                        ". From: " + addressbook.getLocation());
     287                                        logMissing(log, action, key, addressbook);
    290288                                        invalid++;
    291289                                        continue;
     
    315313                                        if (pod2 == null) {
    316314                                            // we didn't have the old name
    317                                             // TODO check inner sig anyway?
     315                                            // check inner sig anyway
     316                                            if (!he.hasValidInnerSig()) {
     317                                                logInner(log, action, key, addressbook);
     318                                                invalid++;
     319                                                continue;
     320                                            }
    318321                                        } else if (pod2.contains(pod)) {
    319322                                            // checks out, so verify the inner sig
    320323                                            if (!he.hasValidInnerSig()) {
    321                                                 if (log != null)
    322                                                     log.append("Action: " + action + " failed because" +
    323                                                                " inner signature for old name " + poldname +
    324                                                                " failed" +
    325                                                                ". From: " + addressbook.getLocation());
     324                                                logInner(log, action, key, addressbook);
    326325                                                invalid++;
    327326                                                continue;
     
    334333                                        }
    335334                                    } else {
    336                                         if (log != null)
    337                                             log.append("Action: " + action + " failed, missing required parameters" +
    338                                                        ". From: " + addressbook.getLocation());
     335                                        logMissing(log, action, key, addressbook);
    339336                                        invalid++;
    340337                                        continue;
     
    350347                                        if (pod2 == null) {
    351348                                            // we didn't have the old name
    352                                             // TODO check inner sig anyway?
     349                                            // check inner sig anyway
     350                                            if (!he.hasValidInnerSig()) {
     351                                                logInner(log, action, key, addressbook);
     352                                                invalid++;
     353                                                continue;
     354                                            }
    353355                                        } else if (pod2.contains(dest)) {
    354356                                            // we already have the new dest
     
    358360                                            // checks out, so verify the inner sig
    359361                                            if (!he.hasValidInnerSig()) {
    360                                                 if (log != null)
    361                                                     log.append("Action: " + action + " failed because" +
    362                                                                " inner signature for key " + key +
    363                                                                " failed" +
    364                                                                ". From: " + addressbook.getLocation());
     362                                                logInner(log, action, key, addressbook);
    365363                                                invalid++;
    366364                                                continue;
     
    383381                                        }
    384382                                    } else {
    385                                         if (log != null)
    386                                             log.append("Action: " + action + " failed, missing required parameters" +
    387                                                        ". From: " + addressbook.getLocation());
     383                                        logMissing(log, action, key, addressbook);
    388384                                        invalid++;
    389385                                        continue;
     
    433429                                        }
    434430                                    } else {
    435                                         if (log != null)
    436                                             log.append("Action: " + action + " failed, missing required parameters" +
    437                                                        ". From: " + addressbook.getLocation());
     431                                        logMissing(log, action, key, addressbook);
    438432                                        invalid++;
    439433                                        continue;
     
    528522                                        }
    529523                                    } else {
    530                                         if (log != null)
    531                                             log.append("Action: " + action + " failed, missing required parameters" +
    532                                                        ". From: " + addressbook.getLocation());
     524                                        logMissing(log, action, "delete", addressbook);
    533525                                        invalid++;
    534526                                    }
     
    604596                                        }
    605597                                    } else {
    606                                         if (log != null)
    607                                             log.append("Action: " + action + " failed, missing required parameters" +
    608                                                        ". From: " + addressbook.getLocation());
     598                                        logMissing(log, action, "delete", addressbook);
    609599                                        invalid++;
    610600                                    }
     
    667657    }
    668658
     659    /** @since 0.9.26 */
     660    private static void logInner(Log log, String action, String name, AddressBook addressbook) {
     661        if (log != null) {
     662            log.append("Action: " + action + " failed because" +
     663                       " inner signature for key " + name +
     664                       " failed" +
     665                       ". From: " + addressbook.getLocation());
     666        }
     667    }
     668
     669    /** @since 0.9.26 */
     670    private static void logMissing(Log log, String action, String name, AddressBook addressbook) {
     671        if (log != null) {
     672            log.append("Action: " + action + " for " + name +
     673                       " failed, missing required parameters" +
     674                       ". From: " + addressbook.getLocation());
     675        }
     676    }
     677
     678    /** @since 0.9.26 */
    669679    private static void logMismatch(Log log, String action, String name, List<Destination> dests,
    670680                                    String olddest, AddressBook addressbook) {
Note: See TracChangeset for help on using the changeset viewer.