Test API "This request is only accepted using POST!" Fix

PoC-Consortium · Jan 2, 2019 · 0cae253 · 0cae253
1 parent f9e79fe
commit 0cae253
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/src/brs/http/APITag.java b/src/brs/http/APITag.java
@@ -5,7 +5,7 @@ public enum APITag {
   ACCOUNTS("Accounts"), ALIASES("Aliases"), AE("Asset Exchange"), CREATE_TRANSACTION("Create Transaction"),
   BLOCKS("Blocks"), DGS("Digital Goods Store"), FORGING("Forging"), INFO("Server Info"), MESSAGES("Messages"),
   MINING("Mining"), TRANSACTIONS("Transactions"), TOKENS("Tokens"), VS("Voting System"), AT("Automated Transaction"),
-  FEES("Fees"), UTILS("Utils"), DEBUG("Debug");
+  FEES("Fees"), UTILS("Utils"), DEBUG("Debug"), PEER_INFO("Server Peer Info");
 
   private final String displayName;
 

diff --git a/src/brs/http/APITestServlet.java b/src/brs/http/APITestServlet.java
@@ -55,7 +55,7 @@ public class APITestServlet extends HttpServlet {
       + "            }\n"
       + "            $.ajax({\n"
       + "                url: url,\n"
-      + "                type: 'GET',\n"
+      + "                type: 'POST',\n"
       + "            })\n"
       + "            .done(function(result) {\n"
       + "                var resultStr = JSON.stringify(JSON.parse(result), null, 4);\n"

diff --git a/src/brs/http/GetMyPeerInfo.java b/src/brs/http/GetMyPeerInfo.java
@@ -12,7 +12,7 @@ public final class GetMyPeerInfo extends APIServlet.APIRequestHandler {
   private final TransactionProcessor transactionProcessor;
 
   public GetMyPeerInfo(BlockchainProcessor blockchainProcessor, TransactionProcessor transactionProcessor) {
-    super(new APITag[]{APITag.INFO});
+    super(new APITag[]{APITag.PEER_INFO});
     this.blockchainProcessor = blockchainProcessor;
     this.transactionProcessor = transactionProcessor;
   }

diff --git a/src/brs/peer/Peers.java b/src/brs/peer/Peers.java
@@ -834,7 +834,8 @@ private static void feedPeer(Peer peer, Function<Peer, List<Transaction>> foodDi
       if(response != null && response.get("error") == null) {
         doneFeedingLog.accept(peer, transactionsToSend);
       } else {
-        logger.warn("Error feeding {} transactions: {} error: {}", peer.getPeerAddress(), transactionsToSend.stream().map(t -> t.getId()).collect(Collectors.toList()), response);
+        logger.warn("Error feeding {} transactions: {} error: {} - removing peer for now.", peer.getPeerAddress(), transactionsToSend.stream().map(t -> t.getId()).collect(Collectors.toList()), response);
+        peer.remove();
       }
     } else {
       logger.trace("No need to feed {}", peer.getPeerAddress());