Merge pull request #8 from curityio/bugfix/fix-error-message

Updated SDK version and fixed error message

Author: mtrojanowski

README.rst

@@ -8,18 +8,18 @@ GitHub Authenticator Plug-in
     :target: https://curity.io/resources/code-examples/status/
 
 
-This project provides an opens source GitHub Authenticator plug-in for the Curity Identity Server. This allows an administrator to add functionality to Curity which will then enable end users to login using their GitHub credentials. The app that integrates with Curity may also be configured to receive the GitHub access token, allowing it to manage resources in a GitHub.
+This project provides an opens source GitHub Authenticator plug-in for the Curity Identity Server. This allows an administrator to add functionality to the Curity Identity Server which will then enable end users to login using their GitHub credentials. The app that integrates with the Curity Identity Server may also be configured to receive the GitHub access token, allowing it to manage resources in a GitHub.
 
 System Requirements
 ~~~~~~~~~~~~~~~~~~~
 
-* Curity Identity Server 2.4.0 and `its system requirements <https://developer.curity.io/docs/latest/system-admin-guide/system-requirements.html>`_
+* Curity Identity Server 7.1.0 and `its system requirements <https://developer.curity.io/docs/latest/system-admin-guide/system-requirements.html>`_
 
 Requirements for Building from Source
 """""""""""""""""""""""""""""""""""""
 
 * Maven 3
-* Java JDK v. 8
+* Java SDK 17 or later
 
 Compiling the Plug-in from Source
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

pom.xml

@@ -6,7 +6,7 @@
 
     <groupId>io.curity.identityserver.plugin</groupId>
     <artifactId>identityserver.plugins.authenticators.github</artifactId>
-    <version>1.4.0</version>
+    <version>1.5.0</version>
 
     <name>GitHub Authenticator</name>
 
@@ -21,8 +21,8 @@
                 <artifactId>maven-compiler-plugin</artifactId>
                 <version>3.1</version>
                 <configuration>
-                    <source>1.8</source>
-                    <target>1.8</target>
+                    <source>17</source>
+                    <target>17</target>
                 </configuration>
             </plugin>
         </plugins>
@@ -32,12 +32,14 @@
         <dependency>
             <groupId>se.curity.identityserver</groupId>
             <artifactId>identityserver.sdk</artifactId>
-            <version>5.2.0</version>
+            <version>7.1.0</version>
+            <scope>provided</scope>
         </dependency>
         <dependency>
             <groupId>org.slf4j</groupId>
             <artifactId>slf4j-api</artifactId>
-            <version>1.7.25</version>
+            <version>1.7.36</version>
+            <scope>provided</scope>
         </dependency>
     </dependencies>
 

src/main/java/io/curity/identityserver/plugin/github/authentication/CallbackRequestHandler.java

@@ -275,7 +275,7 @@ private void handleError(CallbackGetRequestModel requestModel)
             _logger.warn("Got an error from StackExchange: {} - {}", requestModel.getError(),
                     requestModel.getErrorDescription());
 
-            throw _exceptionFactory.externalServiceException("Login with StackExchange failed");
+            throw _exceptionFactory.externalServiceException("Login with GitHub failed");
         }
     }