Merge pull request #28 from Ferlab-Ste-Justine/fix/cqdg-752_fix_message

fix: CQDG-752 fix unauthorized message

src/main/scala/ca/ferlab/ferload/client/Main.scala

@@ -11,7 +11,7 @@ import picocli.CommandLine.Command
 import java.io.File
 
 @Command(name = "ferload-client", mixinStandardHelpOptions = true,
-  version = Array("2.1.0"),
+  version = Array("2.1.1"),
   description = Array("Official Ferload Client command line interface for files download."),
   subcommands = Array(classOf[Configure], classOf[Download]))
 class Main extends Runnable {

src/main/scala/ca/ferlab/ferload/client/clients/BaseHttpClient.scala

@@ -31,6 +31,13 @@ abstract class BaseHttpClient {
     s"$message, code: $status, message:\n${msg.getOrElse("")}"
   }
 
+  protected def formatUnauthorizedMessage(message: String, body: Option[String]): String = {
+    val msg = body.map(r => new JSONObject(r).get("msg"))
+
+    s"$message\n${msg.getOrElse("")}"
+  }
+
+
   protected def toMap(body: Option[String], lineContents: Seq[LineContent]): Map[LineContent, String] = {
     body.map(new JSONObject(_).toMap.asScala.map({ case (key, value) =>
       lineContents.find(_.filePointer == key).get -> value.toString

src/main/scala/ca/ferlab/ferload/client/clients/FerloadClient.scala

@@ -27,7 +27,7 @@ class FerloadClient(userConfig: UserConfig) extends BaseHttpClient with IFerload
     val (body, status) = executeHttpRequest(httpRequest)
     status match {
       case 200 => toMap(body, manifestContent.lines)
-      case 403 => throw new UnauthorizedException(formatExceptionMessage("No enough access rights to download the following files", status, body))
+      case 403 => throw new UnauthorizedException(formatUnauthorizedMessage("The manifest contains files you don't have access to. Please review the following unauthorized file IDs:", body))
       case _ => throw new IllegalStateException(formatExceptionMessage("Failed to retrieve download link(s)", status, body))
     }
   }