From 17f97ff7bc34a3d8e0a1e56d6f65aa273ac71487 Mon Sep 17 00:00:00 2001
From: Marcus Hill <bcf8c9f2@opayq.com>
Date: Thu, 14 Apr 2022 20:05:15 +0100
Subject: [PATCH] Deduplicate API GET check into single file

---
 api/v1/assets/read.php        | 8 +-------
 api/v1/certificates/read.php  | 8 +-------
 api/v1/contacts/read.php      | 8 +-------
 api/v1/domains/read.php       | 8 +-------
 api/v1/networks/read.php      | 8 +-------
 api/v1/require_get_method.php | 8 ++++++++
 api/v1/software/read.php      | 8 +-------
 api/v1/tickets/read.php       | 8 +-------
 8 files changed, 15 insertions(+), 49 deletions(-)
 create mode 100644 api/v1/require_get_method.php

diff --git a/api/v1/assets/read.php b/api/v1/assets/read.php
index 92b9b3ba..419ecfb1 100644
--- a/api/v1/assets/read.php
+++ b/api/v1/assets/read.php
@@ -1,13 +1,7 @@
 <?php
 require('../validate_api_key.php');
 
-if($_SERVER['REQUEST_METHOD'] !== "GET"){
-    header("HTTP/1.1 405 Method Not Allowed");
-    $return_arr['success'] = "False";
-    $return_arr['message'] = "Can only send GET requests to this endpoint.";
-    echo json_encode($return_arr);
-    exit();
-}
+require('../require_get_method.php');
 
 // Asset via ID (single)
 if(isset($_GET['asset_id'])){
diff --git a/api/v1/certificates/read.php b/api/v1/certificates/read.php
index 01f3baa5..f07ef6bc 100644
--- a/api/v1/certificates/read.php
+++ b/api/v1/certificates/read.php
@@ -1,13 +1,7 @@
 <?php
 require('../validate_api_key.php');
 
-if($_SERVER['REQUEST_METHOD'] !== "GET"){
-    header("HTTP/1.1 405 Method Not Allowed");
-    $return_arr['success'] = "False";
-    $return_arr['message'] = "Can only send GET requests to this endpoint.";
-    echo json_encode($return_arr);
-    exit();
-}
+require('../require_get_method.php');
 
 // Specific certificate via ID (single)
 if(isset($_GET['certificate_id'])){
diff --git a/api/v1/contacts/read.php b/api/v1/contacts/read.php
index 7f7b4903..72bf3314 100644
--- a/api/v1/contacts/read.php
+++ b/api/v1/contacts/read.php
@@ -1,13 +1,7 @@
 <?php
 require('../validate_api_key.php');
 
-if($_SERVER['REQUEST_METHOD'] !== "GET"){
-    header("HTTP/1.1 405 Method Not Allowed");
-    $return_arr['success'] = "False";
-    $return_arr['message'] = "Can only send GET requests to this endpoint.";
-    echo json_encode($return_arr);
-    exit();
-}
+require('../require_get_method.php');
 
 // Specific contact via ID (single)
 if(isset($_GET['contact_id'])){
diff --git a/api/v1/domains/read.php b/api/v1/domains/read.php
index 7beae32c..087ab7a4 100644
--- a/api/v1/domains/read.php
+++ b/api/v1/domains/read.php
@@ -1,13 +1,7 @@
 <?php
 require('../validate_api_key.php');
 
-if($_SERVER['REQUEST_METHOD'] !== "GET"){
-    header("HTTP/1.1 405 Method Not Allowed");
-    $return_arr['success'] = "False";
-    $return_arr['message'] = "Can only send GET requests to this endpoint.";
-    echo json_encode($return_arr);
-    exit();
-}
+require('../require_get_method.php');
 
 // Specific domain via ID (single)
 if(isset($_GET['domain_id'])){
diff --git a/api/v1/networks/read.php b/api/v1/networks/read.php
index 7761f8e9..d0f6bb33 100644
--- a/api/v1/networks/read.php
+++ b/api/v1/networks/read.php
@@ -1,13 +1,7 @@
 <?php
 require('../validate_api_key.php');
 
-if($_SERVER['REQUEST_METHOD'] !== "GET"){
-    header("HTTP/1.1 405 Method Not Allowed");
-    $return_arr['success'] = "False";
-    $return_arr['message'] = "Can only send GET requests to this endpoint.";
-    echo json_encode($return_arr);
-    exit();
-}
+require('../require_get_method.php');
 
 // Specific network via ID (single)
 if(isset($_GET['network_id'])){
diff --git a/api/v1/require_get_method.php b/api/v1/require_get_method.php
new file mode 100644
index 00000000..a6391c9f
--- /dev/null
+++ b/api/v1/require_get_method.php
@@ -0,0 +1,8 @@
+<?php
+if($_SERVER['REQUEST_METHOD'] !== "GET"){
+  header("HTTP/1.1 405 Method Not Allowed");
+  $return_arr['success'] = "False";
+  $return_arr['message'] = "Can only send GET requests to this endpoint.";
+  echo json_encode($return_arr);
+  exit();
+}
\ No newline at end of file
diff --git a/api/v1/software/read.php b/api/v1/software/read.php
index b82514d1..52b16404 100644
--- a/api/v1/software/read.php
+++ b/api/v1/software/read.php
@@ -1,13 +1,7 @@
 <?php
 require('../validate_api_key.php');
 
-if($_SERVER['REQUEST_METHOD'] !== "GET"){
-    header("HTTP/1.1 405 Method Not Allowed");
-    $return_arr['success'] = "False";
-    $return_arr['message'] = "Can only send GET requests to this endpoint.";
-    echo json_encode($return_arr);
-    exit();
-}
+require('../require_get_method.php');
 
 // Specific software via ID (single)
 if(isset($_GET['software_id'])){
diff --git a/api/v1/tickets/read.php b/api/v1/tickets/read.php
index 7366dd99..74ccfa0e 100644
--- a/api/v1/tickets/read.php
+++ b/api/v1/tickets/read.php
@@ -1,13 +1,7 @@
 <?php
 require('../validate_api_key.php');
 
-if($_SERVER['REQUEST_METHOD'] !== "GET"){
-  header("HTTP/1.1 405 Method Not Allowed");
-  $return_arr['success'] = "False";
-  $return_arr['message'] = "Can only send GET requests to this endpoint.";
-  echo json_encode($return_arr);
-  exit();
-}
+require('../require_get_method.php');
 
 // Specific ticket via ID (single)
 if(isset($_GET['ticket_id'])){