no message

Author: jasonbahl

README.md

@@ -6,26 +6,26 @@ JSON Web Tokens are an open, industry standard [RFC 7519](https://tools.ietf.org
 
 This plugin is based off the `wp-api-jwt-auth` plugin by Enrique Chavez (https://github.com/Tmeister), but modified specifically for use with the <a href="https://github.com/wp-graphql/wp-graphql" target="_blank">WPGraphQL</a> plugin.
 
-### IN ACTION
+## IN ACTION
 
 ![WPGraphQL JWT AUTH DEMO - LOGIN, RECEIVE TOKEN AND USER DATA](https://github.com/wp-graphql/wp-graphql-jwt-authentication/blob/master/wpgraphql-auth.gif?raw=true "LOGIN, RECEIVE TOKEN AND USER DATA")
 
-### REQUIREMENTS
+## REQUIREMENTS
 
-### WPGraphQL
+#### WPGraphQL
 
 This plugin exists as an extension of <a href="https://github.com/wp-graphql/wp-graphql" target="_blank">WPGraphQL</a> and requires that plugin to be active.
 
-### PHP
+#### PHP
 
-**Minimum PHP version: 5.3**
+**Minimum PHP version: 5.5**
 
-### PHP HTTP Authorization Header enabled
+#### PHP HTTP Authorization Header enabled
 
 Check with your hosting provider to determine if the Authorization headers are supported or how to add support if they aren't.
 
 ### CONFIGURATION & ACTIVATION
-### Configure the Secret Key
+#### Configure the Secret Key
 
 The JWT needs a **secret key** to sign the token this **secret key** must be unique and never revealed.
 
@@ -37,14 +37,5 @@ define('GRAPHQL_JWT_AUTH_SECRET_KEY', 'your-top-secrect-key');
 
 You can use a string from here https://api.wordpress.org/secret-key/1.1/salt/
 
-### Configurate CORs Support
-
-The **wp-graphql-jwt-authentication** plugin has the option to activate [CORs](https://en.wikipedia.org/wiki/Cross-origin_resource_sharing) support.
-
-To enable the CORs Support edit your wp-config.php file and add a new constant called **GRAPHQL_JWT_AUTH_CORS_ENABLE**
-
-`
-define('JWT_AUTH_CORS_ENABLE', true);
-`
 
 Finally activate the plugin within your wp-admin.

src/Config.php

@@ -31,11 +31,6 @@ public function __construct() {
 		 */
 		self::$secret_key = self::get_secret_key();
 
-		/**
-		 * Add cors support to the header if enabled
-		 */
-		add_action( 'graphql_process_http_request', [ $this, 'add_cors_support' ] );
-
 		/**
 		 * Determine the current user
 		 */
@@ -81,31 +76,6 @@ public static function get_secret_key() {
 
 	}
 
-	/**
-	 * If enabled, this will add "cors" support by adjusting the headers for the GraphQL request. This first
-	 * checks to see if the constant is defined, then passes it through a filter, allowing for that value to be
-	 * dynamic, for example an admin UI could be created that would allow for the option to be checked.
-	 *
-	 * @since 0.0.1
-	 * @return void
-	 */
-	public function add_cors_support() {
-
-		/**
-		 * Cors is disabled by default, but can be defined / filtered to be enabled
-		 *
-		 * @since 0.0.1
-		 */
-		$enable_cors = defined( 'GRAPHQL_JWT_AUTH_CORS_ENABLE' ) ? GRAPHQL_JWT_AUTH_CORS_ENABLE : false;
-		$enable_cors = apply_filters( 'graphql_jwt_auth_enable_cors', $enable_cors );
-
-		if ( $enable_cors ) {
-			$headers = apply_filters( 'graphql_jwt_auth_cors_allow_headers', 'Access-Control-Allow-Headers, Content-Type, Authorization' );
-			header( sprintf( 'Access-Control-Allow-Headers: %s', $headers ) );
-		}
-
-	}
-
 	/**
 	 * This is our Middleware to try to authenticate the user according to the
 	 * token send.

tests/test-authentication.php

@@ -57,7 +57,7 @@ public function testLoginWithPage() {
 		 */
 		$query = '
 		mutation { 
-			login(username: "testUser", password: "testPassword" ) {  
+			login( input: { username: "testUser", password: "testPassword" } ) {  
 				user {
 					username
 					pages{
@@ -78,8 +78,6 @@ public function testLoginWithPage() {
 		 */
 		$actual = do_graphql_request( $query );
 
-		var_dump( $actual['data']['login']['user']['pages']['edges'][0]['node'] );
-
 		/**
 		 * Establish the expectation for the output of the query
 		 */