remove unused legacy endpoints

web/src/main/java/de/cotto/lndmanagej/controller/LegacyController.java

@@ -6,9 +6,7 @@ import de.cotto.lndmanagej.model.Channel;
 import de.cotto.lndmanagej.model.ChannelId;
 import de.cotto.lndmanagej.model.LocalOpenChannel;
 import de.cotto.lndmanagej.model.Pubkey;
-import de.cotto.lndmanagej.service.BalanceService;
 import de.cotto.lndmanagej.service.ChannelService;
-import de.cotto.lndmanagej.service.FeeService;
 import de.cotto.lndmanagej.service.NodeService;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
@@ -17,7 +15,6 @@ import org.springframework.web.bind.annotation.RestController;
 
 import java.util.Comparator;
 import java.util.stream.Collectors;
-import java.util.stream.Stream;
 
 @RestController
 @RequestMapping("/legacy")
@@ -25,21 +22,15 @@ public class LegacyController {
     private static final String NEWLINE = "\n";
     private final NodeService nodeService;
     private final ChannelService channelService;
-    private final FeeService feeService;
-    private final BalanceService balanceService;
     private final Metrics metrics;
 
     public LegacyController(
             NodeService nodeService,
             ChannelService channelService,
-            FeeService feeService,
-            BalanceService balanceService,
             Metrics metrics
     ) {
         this.nodeService = nodeService;
         this.channelService = channelService;
-        this.feeService = feeService;
-        this.balanceService = balanceService;
         this.metrics = metrics;
     }
 
@@ -53,14 +44,6 @@ public class LegacyController {
                 .collect(Collectors.joining(NEWLINE));
     }
 
-    @GetMapping("/open-channels")
-    public String getOpenChannelIds() {
-        mark("getOpenChannelIds");
-        return getOpenChannelIdsSorted()
-                .map(ChannelId::toString)
-                .collect(Collectors.joining(NEWLINE));
-    }
-
     @GetMapping("/open-channels/pretty")
     public String getOpenChannelIdsPretty() {
         mark("getOpenChannelIdsPretty");
@@ -76,24 +59,6 @@ public class LegacyController {
                 .collect(Collectors.joining(NEWLINE));
     }
 
-    @GetMapping("/closed-channels")
-    public String getClosedChannelIds() {
-        mark("getClosedChannelIds");
-        return getClosedChannelIdsSorted()
-                .map(ChannelId::toString)
-                .collect(Collectors.joining(NEWLINE));
-    }
-
-    @GetMapping("/force-closing-channels")
-    public String getForceClosingChannelIds() {
-        mark("getForceClosingChannelIds");
-        return channelService.getForceClosingChannels().stream()
-                .map(Channel::getId)
-                .sorted()
-                .map(ChannelId::toString)
-                .collect(Collectors.joining(NEWLINE));
-    }
-
     @GetMapping("/peer-pubkeys")
     public String getPeerPubkeys() {
         mark("getPeerPubkeys");
@@ -105,66 +70,6 @@ public class LegacyController {
                 .collect(Collectors.joining(NEWLINE));
     }
 
-    @GetMapping("/channel/{channelId}/incoming-fee-rate")
-    public long getIncomingFeeRate(@PathVariable ChannelId channelId) {
-        mark("getIncomingFeeRate");
-        return feeService.getFeeConfiguration(channelId).incomingFeeRate();
-    }
-
-    @GetMapping("/channel/{channelId}/outgoing-fee-rate")
-    public long getOutgoingFeeRate(@PathVariable ChannelId channelId) {
-        mark("getOutgoingFeeRate");
-        return feeService.getFeeConfiguration(channelId).outgoingFeeRate();
-    }
-
-    @GetMapping("/channel/{channelId}/incoming-base-fee")
-    public long getIncomingBaseFee(@PathVariable ChannelId channelId) {
-        mark("getIncomingBaseFee");
-        return feeService.getFeeConfiguration(channelId).incomingBaseFee().milliSatoshis();
-    }
-
-    @GetMapping("/channel/{channelId}/outgoing-base-fee")
-    public long getOutgoingBaseFee(@PathVariable ChannelId channelId) {
-        mark("getOutgoingBaseFee");
-        return feeService.getFeeConfiguration(channelId).outgoingBaseFee().milliSatoshis();
-    }
-
-    @GetMapping("/channel/{channelId}/available-local-balance")
-    public long getAvailableLocalBalanceForChannel(@PathVariable ChannelId channelId) {
-        mark("getAvailableLocalBalanceForChannel");
-        return balanceService.getAvailableLocalBalance(channelId).satoshis();
-    }
-
-    @GetMapping("/channel/{channelId}/available-remote-balance")
-    public long getAvailableRemoteBalanceForChannel(@PathVariable ChannelId channelId) {
-        mark("getAvailableRemoteBalanceForChannel");
-        return balanceService.getAvailableRemoteBalance(channelId).satoshis();
-    }
-
-    @GetMapping("/node/{pubkey}/available-local-balance")
-    public long getAvailableLocalBalanceForPeer(@PathVariable Pubkey pubkey) {
-        mark("getAvailableLocalBalanceForPeer");
-        return balanceService.getAvailableLocalBalance(pubkey).satoshis();
-    }
-
-    @GetMapping("/node/{pubkey}/available-remote-balance")
-    public long getAvailableRemoteBalanceForPeer(@PathVariable Pubkey pubkey) {
-        mark("getAvailableRemoteBalanceForPeer");
-        return balanceService.getAvailableRemoteBalance(pubkey).satoshis();
-    }
-
-    private Stream<ChannelId> getOpenChannelIdsSorted() {
-        return channelService.getOpenChannels().stream()
-                .map(Channel::getId)
-                .sorted();
-    }
-
-    private Stream<ChannelId> getClosedChannelIdsSorted() {
-        return channelService.getClosedChannels().stream()
-                .map(Channel::getId)
-                .sorted();
-    }
-
     private void mark(String name) {
         metrics.mark(MetricRegistry.name(getClass(), name));
     }