Encode logfile option with URLEncoder (#1363)

* Encode logfile option with URLEncoder

* Add fallback to logfile

Author: jasonxh

codegen/src/main/scala/akka/grpc/gen/Main.scala

@@ -5,6 +5,7 @@
 package akka.grpc.gen
 
 import java.io.ByteArrayOutputStream
+import java.net.URLDecoder
 
 import com.google.protobuf.compiler.PluginProtos.CodeGeneratorRequest
 import akka.grpc.gen.javadsl.{ JavaClientCodeGenerator, JavaInterfaceCodeGenerator, JavaServerCodeGenerator }
@@ -45,7 +46,14 @@ object Main extends App {
   private val extraGenerators: List[String] =
     parameters.getOrElse("extra_generators", "").split(";").toList.filter(_ != "")
 
-  private val logger = parameters.get("logfile").map(new FileLogger(_)).getOrElse(SilencedLogger)
+  // Prefer logfile_enc with fallback to logfile
+  private val logger: Logger =
+    parameters
+      .get("logfile_enc")
+      .map(URLDecoder.decode(_, "utf-8"))
+      .orElse(parameters.get("logfile"))
+      .map(new FileLogger(_))
+      .getOrElse(SilencedLogger)
 
   val out = {
     val codeGenerators =

gradle-plugin/src/main/groovy/akka/grpc/gradle/AkkaGrpcPlugin.groovy

@@ -11,6 +11,7 @@ import org.gradle.internal.component.local.model.DefaultProjectComponentSelector
 import org.gradle.util.GradleVersion
 import org.gradle.util.VersionNumber
 
+import java.net.URLEncoder
 import java.nio.file.Files
 import java.nio.file.Path
 
@@ -125,7 +126,7 @@ class AkkaGrpcPlugin implements Plugin<Project> {
                             option "server_power_apis=${akkaGrpcExt.serverPowerApis}"
                             option "use_play_actions=${akkaGrpcExt.usePlayActions}"
                             option "extra_generators=${akkaGrpcExt.extraGenerators.join(';')}"
-                            option "logfile=${project.projectDir.toPath().relativize(logFile).toString()}"
+                            option "logfile_enc=${URLEncoder.encode(logFile.toString(), "utf-8")}"
                             if (akkaGrpcExt.includeStdTypes) {
                                 option "include_std_types=true"
                             }