Ignore:
Timestamp:
2017-08-22T22:26:32+02:00 (7 years ago)
Author:
Don-vip
Message:

see #15182 - deprecate all Main logging methods and introduce suitable replacements in Logging for most of them

Location:
trunk/src/org/openstreetmap/josm/io/remotecontrol
Files:
10 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RemoteControlHttpServer.java

    r11620 r12620  
    1010
    1111import org.openstreetmap.josm.Main;
     12import org.openstreetmap.josm.tools.Logging;
    1213
    1314/**
    … …  
    3738            instance4.start();
    3839        } catch (IOException ex) {
    39             Main.debug(ex);
    40             Main.warn(marktr("Cannot start IPv4 remotecontrol server on port {0}: {1}"),
     40            Logging.debug(ex);
     41            Logging.warn(marktr("Cannot start IPv4 remotecontrol server on port {0}: {1}"),
    4142                    Integer.toString(port), ex.getLocalizedMessage());
    4243        }
    … …  
    4748            /* only show error when we also have no IPv4 */
    4849            if (instance4 == null) {
    49                 Main.debug(ex);
    50                 Main.warn(marktr("Cannot start IPv6 remotecontrol server on port {0}: {1}"),
     50                Logging.debug(ex);
     51                Logging.warn(marktr("Cannot start IPv6 remotecontrol server on port {0}: {1}"),
    5152                    Integer.toString(port), ex.getLocalizedMessage());
    5253            }
    … …  
    6364                instance4.stopServer();
    6465            } catch (IOException ioe) {
    65                 Main.error(ioe);
     66                Logging.error(ioe);
    6667            }
    6768            instance4 = null;
    … …  
    7172                instance6.stopServer();
    7273            } catch (IOException ioe) {
    73                 Main.error(ioe);
     74                Logging.error(ioe);
    7475            }
    7576            instance6 = null;
    … …  
    9697    @Override
    9798    public void run() {
    98         Main.info(marktr("RemoteControl::Accepting remote connections on {0}:{1}"),
     99        Logging.info(marktr("RemoteControl::Accepting remote connections on {0}:{1}"),
    99100                server.getInetAddress(), Integer.toString(server.getLocalPort()));
    100101        while (true) {
    … …  
    105106            } catch (SocketException e) {
    106107                if (!server.isClosed()) {
    107                     Main.error(e);
     108                    Logging.error(e);
    108109                } else {
    109110                    // stop the thread automatically if server is stopped
    … …  
    111112                }
    112113            } catch (IOException ioe) {
    113                 Main.error(ioe);
     114                Logging.error(ioe);
    114115            }
    115116        }
    … …  
    122123     */
    123124    public void stopServer() throws IOException {
    124         Main.info(marktr("RemoteControl::Server {0}:{1} stopped."),
     125        Logging.info(marktr("RemoteControl::Server {0}:{1} stopped."),
    125126        server.getInetAddress(), Integer.toString(server.getLocalPort()));
    126127        server.close();

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RemoteControlHttpsServer.java

    r12458 r12620  
    4141import org.openstreetmap.josm.Main;
    4242import org.openstreetmap.josm.data.preferences.StringProperty;
     43import org.openstreetmap.josm.tools.Logging;
    4344
    4445import sun.security.util.ObjectIdentifier;
    … …  
    208209
    209210        if (!path.toFile().exists()) {
    210             Main.debug("No keystore found, creating a new one");
     211            Logging.debug("No keystore found, creating a new one");
    211212
    212213            // Create new keystore like previous one generated with JDK keytool as follows:
    … …  
    252253            ks.load(in, KEYSTORE_PASSWORD.get().toCharArray());
    253254
    254             if (Main.isDebugEnabled()) {
     255            if (Logging.isDebugEnabled()) {
    255256                for (Enumeration<String> aliases = ks.aliases(); aliases.hasMoreElements();) {
    256                     Main.debug("Alias in JOSM keystore: "+aliases.nextElement());
     257                    Logging.debug("Alias in JOSM keystore: {0}", aliases.nextElement());
    257258                }
    258259            }
    … …  
    278279        sslContext.init(kmf.getKeyManagers(), tmf.getTrustManagers(), null);
    279280
    280         if (Main.isTraceEnabled()) {
    281             Main.trace("SSL Context protocol: " + sslContext.getProtocol());
    282             Main.trace("SSL Context provider: " + sslContext.getProvider());
     281        if (Logging.isTraceEnabled()) {
     282            Logging.trace("SSL Context protocol: {0}", sslContext.getProtocol());
     283            Logging.trace("SSL Context provider: {0}", sslContext.getProvider());
    283284        }
    284285
    … …  
    316317                instance4.start();
    317318            } catch (IOException | GeneralSecurityException ex) {
    318                 Main.debug(ex);
    319                 Main.warn(marktr("Cannot start IPv4 remotecontrol https server on port {0}: {1}"),
     319                Logging.debug(ex);
     320                Logging.warn(marktr("Cannot start IPv4 remotecontrol https server on port {0}: {1}"),
    320321                        Integer.toString(port), ex.getLocalizedMessage());
    321322            }
    … …  
    326327                /* only show error when we also have no IPv4 */
    327328                if (instance4 == null) {
    328                     Main.debug(ex);
    329                     Main.warn(marktr("Cannot start IPv6 remotecontrol https server on port {0}: {1}"),
     329                    Logging.debug(ex);
     330                    Logging.warn(marktr("Cannot start IPv6 remotecontrol https server on port {0}: {1}"),
    330331                        Integer.toString(port), ex.getLocalizedMessage());
    331332                }
    … …  
    342343                instance4.stopServer();
    343344            } catch (IOException ioe) {
    344                 Main.error(ioe);
     345                Logging.error(ioe);
    345346            }
    346347            instance4 = null;
    … …  
    350351                instance6.stopServer();
    351352            } catch (IOException ioe) {
    352                 Main.error(ioe);
     353                Logging.error(ioe);
    353354            }
    354355            instance6 = null;
    … …  
    372373        // Create SSL Server factory
    373374        SSLServerSocketFactory factory = sslContext.getServerSocketFactory();
    374         if (Main.isTraceEnabled()) {
    375             Main.trace("SSL factory - Supported Cipher suites: "+Arrays.toString(factory.getSupportedCipherSuites()));
     375        if (Logging.isTraceEnabled()) {
     376            Logging.trace("SSL factory - Supported Cipher suites: {0}", Arrays.toString(factory.getSupportedCipherSuites()));
    376377        }
    377378
    … …  
    379380            RemoteControl.getInet6Address() : RemoteControl.getInet4Address());
    380381
    381         if (Main.isTraceEnabled() && server instanceof SSLServerSocket) {
     382        if (Logging.isTraceEnabled() && server instanceof SSLServerSocket) {
    382383            SSLServerSocket sslServer = (SSLServerSocket) server;
    383             Main.trace("SSL server - Enabled Cipher suites: "+Arrays.toString(sslServer.getEnabledCipherSuites()));
    384             Main.trace("SSL server - Enabled Protocols: "+Arrays.toString(sslServer.getEnabledProtocols()));
    385             Main.trace("SSL server - Enable Session Creation: "+sslServer.getEnableSessionCreation());
    386             Main.trace("SSL server - Need Client Auth: "+sslServer.getNeedClientAuth());
    387             Main.trace("SSL server - Want Client Auth: "+sslServer.getWantClientAuth());
    388             Main.trace("SSL server - Use Client Mode: "+sslServer.getUseClientMode());
     384            Logging.trace("SSL server - Enabled Cipher suites: {0}", Arrays.toString(sslServer.getEnabledCipherSuites()));
     385            Logging.trace("SSL server - Enabled Protocols: {0}", Arrays.toString(sslServer.getEnabledProtocols()));
     386            Logging.trace("SSL server - Enable Session Creation: {0}", sslServer.getEnableSessionCreation());
     387            Logging.trace("SSL server - Need Client Auth: {0}", sslServer.getNeedClientAuth());
     388            Logging.trace("SSL server - Want Client Auth: {0}", sslServer.getWantClientAuth());
     389            Logging.trace("SSL server - Use Client Mode: {0}", sslServer.getUseClientMode());
    389390        }
    390391    }
    … …  
    395396    @Override
    396397    public void run() {
    397         Main.info(marktr("RemoteControl::Accepting secure remote connections on {0}:{1}"),
     398        Logging.info(marktr("RemoteControl::Accepting secure remote connections on {0}:{1}"),
    398399                server.getInetAddress(), Integer.toString(server.getLocalPort()));
    399400        while (true) {
    … …  
    401402                @SuppressWarnings("resource")
    402403                Socket request = server.accept();
    403                 if (Main.isTraceEnabled() && request instanceof SSLSocket) {
     404                if (Logging.isTraceEnabled() && request instanceof SSLSocket) {
    404405                    SSLSocket sslSocket = (SSLSocket) request;
    405                     Main.trace("SSL socket - Enabled Cipher suites: "+Arrays.toString(sslSocket.getEnabledCipherSuites()));
    406                     Main.trace("SSL socket - Enabled Protocols: "+Arrays.toString(sslSocket.getEnabledProtocols()));
    407                     Main.trace("SSL socket - Enable Session Creation: "+sslSocket.getEnableSessionCreation());
    408                     Main.trace("SSL socket - Need Client Auth: "+sslSocket.getNeedClientAuth());
    409                     Main.trace("SSL socket - Want Client Auth: "+sslSocket.getWantClientAuth());
    410                     Main.trace("SSL socket - Use Client Mode: "+sslSocket.getUseClientMode());
    411                     Main.trace("SSL socket - Session: "+sslSocket.getSession());
     406                    Logging.trace("SSL socket - Enabled Cipher suites: {0}", Arrays.toString(sslSocket.getEnabledCipherSuites()));
     407                    Logging.trace("SSL socket - Enabled Protocols: {0}", Arrays.toString(sslSocket.getEnabledProtocols()));
     408                    Logging.trace("SSL socket - Enable Session Creation: {0}", sslSocket.getEnableSessionCreation());
     409                    Logging.trace("SSL socket - Need Client Auth: {0}", sslSocket.getNeedClientAuth());
     410                    Logging.trace("SSL socket - Want Client Auth: {0}", sslSocket.getWantClientAuth());
     411                    Logging.trace("SSL socket - Use Client Mode: {0}", sslSocket.getUseClientMode());
     412                    Logging.trace("SSL socket - Session: {0}", sslSocket.getSession());
    412413                }
    413414                RequestProcessor.processRequest(request);
    414415            } catch (SocketException e) {
    415416                if (!server.isClosed()) {
    416                     Main.error(e);
     417                    Logging.error(e);
    417418                }
    418419            } catch (IOException ioe) {
    419                 Main.error(ioe);
     420                Logging.error(ioe);
    420421            }
    421422        }
    … …  
    428429     */
    429430    public void stopServer() throws IOException {
    430         Main.info(marktr("RemoteControl::Server {0}:{1} stopped."),
     431        Logging.info(marktr("RemoteControl::Server {0}:{1} stopped."),
    431432        server.getInetAddress(), Integer.toString(server.getLocalPort()));
    432433        server.close();

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RequestProcessor.java

    r11952 r12620  
    2525import java.util.regex.Pattern;
    2626
    27 import org.openstreetmap.josm.Main;
    2827import org.openstreetmap.josm.gui.help.HelpUtil;
    2928import org.openstreetmap.josm.io.remotecontrol.handler.AddNodeHandler;
    … …  
    4140import org.openstreetmap.josm.io.remotecontrol.handler.RequestHandler.RequestHandlerForbiddenException;
    4241import org.openstreetmap.josm.io.remotecontrol.handler.VersionHandler;
     42import org.openstreetmap.josm.tools.Logging;
    4343import org.openstreetmap.josm.tools.Utils;
    4444
    … …  
    119119        String commandWithSlash = '/' + command;
    120120        if (handlers.get(commandWithSlash) != null) {
    121             Main.info("RemoteControl: ignoring duplicate command " + command
     121            Logging.info("RemoteControl: ignoring duplicate command " + command
    122122                    + " with handler " + handler.getName());
    123123        } else {
    124124            if (!silent) {
    125                 Main.info("RemoteControl: adding command \"" +
     125                Logging.info("RemoteControl: adding command \"" +
    126126                    command + "\" (handled by " + handler.getSimpleName() + ')');
    127127            }
    … …  
    161161                return;
    162162            }
    163             Main.info("RemoteControl received: " + get);
     163            Logging.info("RemoteControl received: " + get);
    164164
    165165            StringTokenizer st = new StringTokenizer(get);
    … …  
    247247                    out.flush();
    248248                } catch (RequestHandlerErrorException ex) {
    249                     Main.debug(ex);
     249                    Logging.debug(ex);
    250250                    sendError(out);
    251251                } catch (RequestHandlerBadRequestException ex) {
    252                     Main.debug(ex);
     252                    Logging.debug(ex);
    253253                    sendBadRequest(out, ex.getMessage());
    254254                } catch (RequestHandlerForbiddenException ex) {
    255                     Main.debug(ex);
     255                    Logging.debug(ex);
    256256                    sendForbidden(out, ex.getMessage());
    257257                }
    … …  
    259259
    260260        } catch (IOException ioe) {
    261             Main.debug(Main.getErrorMessage(ioe));
     261            Logging.debug(Logging.getErrorMessage(ioe));
    262262        } catch (ReflectiveOperationException e) {
    263             Main.error(e);
     263            Logging.error(e);
    264264            try {
    265265                sendError(out);
    266266            } catch (IOException e1) {
    267                 Main.warn(e1);
     267                Logging.warn(e1);
    268268            }
    269269        } finally {
    … …  
    271271                request.close();
    272272            } catch (IOException e) {
    273                 Main.debug(Main.getErrorMessage(e));
     273                Logging.debug(Logging.getErrorMessage(e));
    274274            }
    275275        }
    … …  
    402402                handler = handlers.get(cmd).getConstructor().newInstance();
    403403            } catch (ReflectiveOperationException ex) {
    404                 Main.error(ex);
     404                Logging.error(ex);
    405405                return null;
    406406            }
    … …  
    410410            return w.toString();
    411411        } catch (IOException e) {
    412             Main.error(e);
     412            Logging.error(e);
    413413            return null;
    414414        }

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/AddNodeHandler.java

    r10716 r12620  
    1717import org.openstreetmap.josm.io.remotecontrol.AddTagsDialog;
    1818import org.openstreetmap.josm.io.remotecontrol.PermissionPrefWithDefault;
     19import org.openstreetmap.josm.tools.Logging;
    1920
    2021/**
    … …  
    7778
    7879        // Parse the arguments
    79         Main.info("Adding node at (" + lat + ", " + lon + ')');
     80        Logging.info("Adding node at (" + lat + ", " + lon + ')');
    8081
    8182        // Create a new node

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/FeaturesHandler.java

    r11567 r12620  
    44import static org.openstreetmap.josm.tools.I18n.tr;
    55
    6 import org.openstreetmap.josm.Main;
    76import org.openstreetmap.josm.io.remotecontrol.PermissionPrefWithDefault;
    87import org.openstreetmap.josm.io.remotecontrol.RequestProcessor;
     8import org.openstreetmap.josm.tools.Logging;
    99
    1010/**
    … …  
    3737                   buf.append(info);
    3838               } else {
    39                    Main.warn("Unknown handler {0} passed to /features request", s);
     39                   Logging.warn("Unknown handler {0} passed to /features request", s);
    4040               }
    4141            }

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/ImageryHandler.java

    r10728 r12620  
    1313import org.openstreetmap.josm.gui.util.GuiHelper;
    1414import org.openstreetmap.josm.io.remotecontrol.PermissionPrefWithDefault;
     15import org.openstreetmap.josm.tools.Logging;
    1516import org.openstreetmap.josm.tools.Utils;
    1617
    … …  
    7778                imgInfo.setDefaultMinZoom(Integer.parseInt(minZoom));
    7879            } catch (NumberFormatException e) {
    79                 Main.error(e);
     80                Logging.error(e);
    8081            }
    8182        }
    … …  
    8586                imgInfo.setDefaultMaxZoom(Integer.parseInt(maxZoom));
    8687            } catch (NumberFormatException e) {
    87                 Main.error(e);
     88                Logging.error(e);
    8889            }
    8990        }
    … …  
    9798            for (ImageryLayer layer : Main.getLayerManager().getLayersOfType(ImageryLayer.class)) {
    9899                if (layer.getInfo().equals(imgInfo)) {
    99                     Main.info("Imagery layer already exists: "+imgInfo);
     100                    Logging.info("Imagery layer already exists: "+imgInfo);
    100101                    return;
    101102                }
    … …  
    106107                Main.getLayerManager().addLayer(ImageryLayer.create(imgInfo));
    107108            } catch (IllegalArgumentException e) {
    108                 Main.error(e, false);
     109                Logging.log(Logging.LEVEL_ERROR, e);
    109110            }
    110111        });

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/ImportHandler.java

    r11746 r12620  
    1414import org.openstreetmap.josm.actions.downloadtasks.DownloadTask;
    1515import org.openstreetmap.josm.io.remotecontrol.PermissionPrefWithDefault;
     16import org.openstreetmap.josm.tools.Logging;
    1617import org.openstreetmap.josm.tools.Utils;
    1718
    … …  
    3637                // For compatibility reasons with older instances of JOSM, arbitrary choice of DownloadOsmTask
    3738                // As of 2015-04, Overpass Turbo requires this branch of code ...
    38                 Main.debug("Remote control, /import: defaulting to DownloadOsmTask");
     39                Logging.debug("Remote control, /import: defaulting to DownloadOsmTask");
    3940                new DownloadOsmTask().loadUrl(isLoadInNewLayer(), url.toExternalForm(), null);
    4041            } else if (Main.pref.getBoolean("remotecontrol.import.interactive", true)) {
    … …  
    4849            }
    4950        } catch (RuntimeException ex) { // NOPMD
    50             Main.warn("RemoteControl: Error parsing import remote control request:");
    51             Main.error(ex);
     51            Logging.warn("RemoteControl: Error parsing import remote control request:");
     52            Logging.error(ex);
    5253            throw new RequestHandlerErrorException(ex);
    5354        }

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadAndZoomHandler.java

    r12472 r12620  
    2929import org.openstreetmap.josm.io.remotecontrol.AddTagsDialog;
    3030import org.openstreetmap.josm.io.remotecontrol.PermissionPrefWithDefault;
     31import org.openstreetmap.josm.tools.Logging;
    3132import org.openstreetmap.josm.tools.SubclassFilteredCollection;
    3233import org.openstreetmap.josm.tools.Utils;
    … …  
    114115            if (command.equals(myCommand)) {
    115116                if (!PermissionPrefWithDefault.LOAD_DATA.isAllowed()) {
    116                     Main.info("RemoteControl: download forbidden by preferences");
     117                    Logging.info("RemoteControl: download forbidden by preferences");
    117118                } else {
    118119                    Area toDownload = null;
    … …  
    138139                    }
    139140                    if (toDownload != null && toDownload.isEmpty()) {
    140                         Main.info("RemoteControl: no download necessary");
     141                        Logging.info("RemoteControl: no download necessary");
    141142                    } else {
    142143                        Future<?> future = osmTask.download(newLayer, new Bounds(minlat, minlon, maxlat, maxlon),
    … …  
    147148            }
    148149        } catch (RuntimeException ex) { // NOPMD
    149             Main.warn("RemoteControl: Error parsing load_and_zoom remote control request:");
    150             Main.error(ex);
     150            Logging.warn("RemoteControl: Error parsing load_and_zoom remote control request:");
     151            Logging.error(ex);
    151152            throw new RequestHandlerErrorException(ex);
    152153        }
    … …  
    200201                });
    201202            } catch (SearchCompiler.ParseError ex) {
    202                 Main.error(ex);
     203                Logging.error(ex);
    203204                throw new RequestHandlerErrorException(ex);
    204205            }
    … …  
    284285                        toSelect.add(SimplePrimitiveId.fromString(item));
    285286                    } catch (IllegalArgumentException ex) {
    286                         Main.warn(ex, "RemoteControl: invalid selection '" + item + "' ignored");
     287                        Logging.log(Logging.LEVEL_WARN, "RemoteControl: invalid selection '" + item + "' ignored", ex);
    287288                    }
    288289                }

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/LoadObjectHandler.java

    r12472 r12620  
    1818import org.openstreetmap.josm.io.remotecontrol.AddTagsDialog;
    1919import org.openstreetmap.josm.io.remotecontrol.PermissionPrefWithDefault;
     20import org.openstreetmap.josm.tools.Logging;
    2021
    2122/**
    … …  
    5960    protected void handleRequest() throws RequestHandlerErrorException, RequestHandlerBadRequestException {
    6061        if (!PermissionPrefWithDefault.LOAD_DATA.isAllowed()) {
    61             Main.info("RemoteControl: download forbidden by preferences");
     62            Logging.info("RemoteControl: download forbidden by preferences");
    6263        }
    6364        if (!ps.isEmpty()) {
    … …  
    102103                    ps.add(SimplePrimitiveId.fromString(i));
    103104                } catch (IllegalArgumentException e) {
    104                     Main.warn(e, "RemoteControl: invalid selection '"+i+"' ignored.");
     105                    Logging.log(Logging.LEVEL_WARN, "RemoteControl: invalid selection '"+i+"' ignored.", e);
    105106                }
    106107            }

  • trunk/src/org/openstreetmap/josm/io/remotecontrol/handler/RequestHandler.java

    r11567 r12620  
    2020import org.openstreetmap.josm.Main;
    2121import org.openstreetmap.josm.io.remotecontrol.PermissionPrefWithDefault;
     22import org.openstreetmap.josm.tools.Logging;
    2223import org.openstreetmap.josm.tools.Utils;
    2324
    … …  
    148149        if (permissionPref != null && permissionPref.pref != null && !Main.pref.getBoolean(permissionPref.pref, permissionPref.defaultVal)) {
    149150            String err = MessageFormat.format("RemoteControl: ''{0}'' forbidden by preferences", myCommand);
    150             Main.info(err);
     151            Logging.info(err);
    151152            throw new RequestHandlerForbiddenException(err);
    152153        }
    … …  
    228229                if (value == null || value.isEmpty()) {
    229230                    error = true;
    230                     Main.warn('\'' + myCommand + "' remote control request must have '" + key + "' parameter");
     231                    Logging.warn('\'' + myCommand + "' remote control request must have '" + key + "' parameter");
    231232                    missingKeys.add(key);
    232233                }
    … …  
    241242            for (String par: args.keySet()) {
    242243                if (!knownParams.contains(par)) {
    243                     Main.warn("Unknown remote control parameter {0}, skipping it", par);
     244                    Logging.warn("Unknown remote control parameter {0}, skipping it", par);
    244245                }
    245246            }
Note: See TracChangeset for help on using the changeset viewer.