From c48f960f33939a273d1e93409e968191bbdc8d66 Mon Sep 17 00:00:00 2001 From: eikek Date: Sat, 2 Mar 2024 22:54:50 +0100 Subject: [PATCH] Fix compile warnings due to scala update --- .../scala/sharry/backend/BackendApp.scala | 2 +- .../sharry/backend/account/OAccount.scala | 2 +- .../sharry/backend/auth/AuthConfig.scala | 2 +- .../sharry/backend/auth/CommandAuth.scala | 2 +- .../scala/sharry/backend/auth/FixedAuth.scala | 2 +- .../scala/sharry/backend/auth/HttpAuth.scala | 2 +- .../sharry/backend/auth/HttpBasicAuth.scala | 2 +- .../scala/sharry/backend/mail/OMail.scala | 9 +++++-- .../scala/sharry/backend/share/OShare.scala | 8 +++--- .../scala/sharry/backend/signup/OSignup.scala | 2 +- .../main/scala/sharry/common/AccountId.scala | 2 +- .../main/scala/sharry/common/LenientUri.scala | 10 ++++--- .../scala/sharry/restserver/RestServer.scala | 2 +- .../restserver/routes/AccountRoutes.scala | 4 +-- .../scala/sharry/restserver/routes/Conv.scala | 26 +++++++++---------- .../restserver/routes/LoginRoutes.scala | 12 +++++++-- .../sharry/restserver/routes/MailRoutes.scala | 17 +++++++----- .../restserver/routes/NotifyRoutes.scala | 12 ++++----- .../restserver/routes/RegisterRoutes.scala | 16 ++++++------ .../restserver/webapp/TemplateRoutes.scala | 2 +- .../sharry/store/ComputeChecksumConfig.scala | 2 +- .../sharry/store/records/RPublishShare.scala | 2 +- 22 files changed, 81 insertions(+), 59 deletions(-) diff --git a/modules/backend/src/main/scala/sharry/backend/BackendApp.scala b/modules/backend/src/main/scala/sharry/backend/BackendApp.scala index 15a717c4..e8a809bf 100644 --- a/modules/backend/src/main/scala/sharry/backend/BackendApp.scala +++ b/modules/backend/src/main/scala/sharry/backend/BackendApp.scala @@ -67,7 +67,7 @@ object BackendApp { cfg.computeChecksum, cfg.files.defaultStoreConfig, connectEC, - true + runMigration = true ) backend <- create(cfg, store) diff --git a/modules/backend/src/main/scala/sharry/backend/account/OAccount.scala b/modules/backend/src/main/scala/sharry/backend/account/OAccount.scala index b75639ef..f5cc7c98 100644 --- a/modules/backend/src/main/scala/sharry/backend/account/OAccount.scala +++ b/modules/backend/src/main/scala/sharry/backend/account/OAccount.scala @@ -80,7 +80,7 @@ object OAccount { shares <- OptionT.liftF(store.transact(RShare.getAllByAccount(id))) _ <- OptionT.liftF(logger.info(s"Delete ${shares.size} shares of account $id")) _ <- OptionT.liftF( - shares.traverse(shareId => ShareQueries.deleteShare(shareId, false)(store)) + shares.traverse(shareId => ShareQueries.deleteShare(shareId, background = false)(store)) ) _ <- OptionT.liftF(store.transact(RAccount.delete(id))) } yield acc diff --git a/modules/backend/src/main/scala/sharry/backend/auth/AuthConfig.scala b/modules/backend/src/main/scala/sharry/backend/auth/AuthConfig.scala index 520246c4..4125d64d 100644 --- a/modules/backend/src/main/scala/sharry/backend/auth/AuthConfig.scala +++ b/modules/backend/src/main/scala/sharry/backend/auth/AuthConfig.scala @@ -93,7 +93,7 @@ object AuthConfig { def github(clientId: String, clientSecret: String): OAuth = OAuth( Ident.unsafe("github"), - true, + enabled = true, "Github", LenientUri.unsafe("https://github.com/login/oauth/authorize"), LenientUri.unsafe("https://github.com/login/oauth/access_token"), diff --git a/modules/backend/src/main/scala/sharry/backend/auth/CommandAuth.scala b/modules/backend/src/main/scala/sharry/backend/auth/CommandAuth.scala index 6a1594ee..7cd19588 100644 --- a/modules/backend/src/main/scala/sharry/backend/auth/CommandAuth.scala +++ b/modules/backend/src/main/scala/sharry/backend/auth/CommandAuth.scala @@ -24,7 +24,7 @@ final class CommandAuth[F[_]: Async]( Ident.fromString(up.user) match { case Right(login) => def okResult: F[LoginResult] = - AddAccount(login, false, ops) + AddAccount(login, admin = false, ops) .flatMap(accId => AuthToken.user(accId, cfg.serverSecret).map(LoginResult.ok) ) diff --git a/modules/backend/src/main/scala/sharry/backend/auth/FixedAuth.scala b/modules/backend/src/main/scala/sharry/backend/auth/FixedAuth.scala index a49826c7..e161c195 100644 --- a/modules/backend/src/main/scala/sharry/backend/auth/FixedAuth.scala +++ b/modules/backend/src/main/scala/sharry/backend/auth/FixedAuth.scala @@ -35,7 +35,7 @@ final class FixedAuth[F[_]: Async](cfg: AuthConfig, op: OAccount[F]) { } private def addAccount(cfg: AuthConfig.Fixed): F[AccountId] = - AddAccount[F](cfg.user, true, AddAccount.AccountOps.from(op)) + AddAccount[F](cfg.user, admin = true, AddAccount.AccountOps.from(op)) def withPosition: (Int, LoginModule[F]) = (cfg.fixed.order, login) diff --git a/modules/backend/src/main/scala/sharry/backend/auth/HttpAuth.scala b/modules/backend/src/main/scala/sharry/backend/auth/HttpAuth.scala index 5d034c95..368680bd 100644 --- a/modules/backend/src/main/scala/sharry/backend/auth/HttpAuth.scala +++ b/modules/backend/src/main/scala/sharry/backend/auth/HttpAuth.scala @@ -25,7 +25,7 @@ final class HttpAuth[F[_]: Async]( Ident.fromString(up.user) match { case Right(login) => def okResult: F[LoginResult] = - AddAccount(login, false, ops) + AddAccount(login, admin = false, ops) .flatMap(accId => AuthToken.user(accId, cfg.serverSecret).map(LoginResult.ok) ) diff --git a/modules/backend/src/main/scala/sharry/backend/auth/HttpBasicAuth.scala b/modules/backend/src/main/scala/sharry/backend/auth/HttpBasicAuth.scala index ed6b5f77..9c8e2376 100644 --- a/modules/backend/src/main/scala/sharry/backend/auth/HttpBasicAuth.scala +++ b/modules/backend/src/main/scala/sharry/backend/auth/HttpBasicAuth.scala @@ -23,7 +23,7 @@ final class HttpBasicAuth[F[_]: Async]( Ident.fromString(up.user) match { case Right(login) => def okResult: F[LoginResult] = - AddAccount(login, false, ops) + AddAccount(login, admin = false, ops) .flatMap(accId => AuthToken.user(accId, cfg.serverSecret).map(LoginResult.ok) ) diff --git a/modules/backend/src/main/scala/sharry/backend/mail/OMail.scala b/modules/backend/src/main/scala/sharry/backend/mail/OMail.scala index a970db05..ae69a63f 100644 --- a/modules/backend/src/main/scala/sharry/backend/mail/OMail.scala +++ b/modules/backend/src/main/scala/sharry/backend/mail/OMail.scala @@ -82,7 +82,12 @@ object OMail { templates = receivers.map { case (login, mailAddress) => ( mailAddress, - TemplateData(login, baseUrl / shareId.id, false, data.aliasName) + TemplateData( + login, + baseUrl / shareId.id, + password = false, + data.aliasName + ) ) } res <- OptionT.liftF(templates.traverse((send _).tupled)) @@ -118,7 +123,7 @@ object OMail { baseUrl: LenientUri ): F[MailData] = { val tpl = cfg.templates.alias - val td = TemplateData(acc.userLogin, baseUrl / aliasId.id, false, "") + val td = TemplateData(acc.userLogin, baseUrl / aliasId.id, password = false, "") MailData(td.render(tpl.subject), td.render(tpl.body)).pure[F] } diff --git a/modules/backend/src/main/scala/sharry/backend/share/OShare.scala b/modules/backend/src/main/scala/sharry/backend/share/OShare.scala index 38fc35b2..765afe66 100644 --- a/modules/backend/src/main/scala/sharry/backend/share/OShare.scala +++ b/modules/backend/src/main/scala/sharry/backend/share/OShare.scala @@ -358,7 +358,7 @@ object OShare { case AddResult.Success => 1.pure[F] case AddResult.EntityExists(_) => - store.transact(RPublishShare.update(share, true, reuseId)) + store.transact(RPublishShare.update(share, enable = true, reuseId = reuseId)) case AddResult.Failure(ex) => Async[F].raiseError(ex) } @@ -370,7 +370,7 @@ object OShare { } def unpublish(share: Ident, accId: AccountId): OptionT[F, Unit] = { - val remove = store.transact(RPublishShare.update(share, false, true)) + val remove = store.transact(RPublishShare.update(share, enable = false, reuseId = true)) for { _ <- OptionT(store.transact(Queries.checkShare(share, accId))) _ <- OptionT.liftF(remove) @@ -414,7 +414,7 @@ object OShare { def deleteShare(accId: AccountId, share: Ident): OptionT[F, Unit] = for { _ <- OptionT(store.transact(Queries.checkShare(share, accId))) - _ <- OptionT.liftF(Queries.deleteShare(share, true)(store)) + _ <- OptionT.liftF(Queries.deleteShare(share, background = true)(store)) } yield () def setDescription( @@ -473,7 +473,7 @@ object OShare { s"Cleaning up expired share '${share.name.getOrElse("")}' " + s"owned by '${account.login.value.id}' (${share.id.id})" ) *> Queries - .deleteShare(share.id, false)( + .deleteShare(share.id, background = false)( store ) } diff --git a/modules/backend/src/main/scala/sharry/backend/signup/OSignup.scala b/modules/backend/src/main/scala/sharry/backend/signup/OSignup.scala index 1b6fc9a3..5b94007e 100644 --- a/modules/backend/src/main/scala/sharry/backend/signup/OSignup.scala +++ b/modules/backend/src/main/scala/sharry/backend/signup/OSignup.scala @@ -95,7 +95,7 @@ object OSignup { AccountState.Active, data.password, None, - false + admin = false ) res <- OAccount(store).use(_.create(acc)) } yield res diff --git a/modules/common/src/main/scala/sharry/common/AccountId.scala b/modules/common/src/main/scala/sharry/common/AccountId.scala index 0b52cdb0..db618a88 100644 --- a/modules/common/src/main/scala/sharry/common/AccountId.scala +++ b/modules/common/src/main/scala/sharry/common/AccountId.scala @@ -17,7 +17,7 @@ case class AccountId(id: Ident, userLogin: Ident, admin: Boolean, alias: Option[ object AccountId { val empty: AccountId = - AccountId(Ident.empty, Ident.empty, false, None) + AccountId(Ident.empty, Ident.empty, admin = false, None) def parse(str: String): Either[String, AccountId] = { val parts = str.split('/').toList.appended("").take(4) diff --git a/modules/common/src/main/scala/sharry/common/LenientUri.scala b/modules/common/src/main/scala/sharry/common/LenientUri.scala index 042b6d34..51697d70 100644 --- a/modules/common/src/main/scala/sharry/common/LenientUri.scala +++ b/modules/common/src/main/scala/sharry/common/LenientUri.scala @@ -72,7 +72,11 @@ case class LenientUri( .covary[F] .rethrow .flatMap(url => - fs2.io.readInputStream(Sync[F].delay(url.openStream()), chunkSize, true) + fs2.io.readInputStream( + Sync[F].delay(url.openStream()), + chunkSize, + closeAfterUse = true + ) ) def readText[F[_]: Sync](chunkSize: Int): F[String] = @@ -115,7 +119,7 @@ object LenientUri { val isRoot = true val isEmpty = false def /(seg: String): Path = - NonEmptyPath(NonEmptyList.of(seg), false) + NonEmptyPath(NonEmptyList.of(seg), trailingSlash = false) def asString = "/" } case object EmptyPath extends Path { @@ -123,7 +127,7 @@ object LenientUri { val isRoot = false val isEmpty = true def /(seg: String): Path = - NonEmptyPath(NonEmptyList.of(seg), false) + NonEmptyPath(NonEmptyList.of(seg), trailingSlash = false) def asString = "" } case class NonEmptyPath(segs: NonEmptyList[String], trailingSlash: Boolean) diff --git a/modules/restserver/src/main/scala/sharry/restserver/RestServer.scala b/modules/restserver/src/main/scala/sharry/restserver/RestServer.scala index 94b2bdb1..a17a044f 100644 --- a/modules/restserver/src/main/scala/sharry/restserver/RestServer.scala +++ b/modules/restserver/src/main/scala/sharry/restserver/RestServer.scala @@ -82,7 +82,7 @@ object RestServer { ).orNotFound // With Middlewares in place - finalHttpApp = Http4sLogger.httpApp(false, false)(httpApp) + finalHttpApp = Http4sLogger.httpApp(logHeaders = false, logBody = false)(httpApp) } yield finalHttpApp } diff --git a/modules/restserver/src/main/scala/sharry/restserver/routes/AccountRoutes.scala b/modules/restserver/src/main/scala/sharry/restserver/routes/AccountRoutes.scala index dc43db38..02c77147 100644 --- a/modules/restserver/src/main/scala/sharry/restserver/routes/AccountRoutes.scala +++ b/modules/restserver/src/main/scala/sharry/restserver/routes/AccountRoutes.scala @@ -49,9 +49,9 @@ object AccountRoutes { res <- backend.account.delete(id).value resp <- res match { case Some(_) => - Ok(BasicResult(true, "Account successfully deleted.")) + Ok(BasicResult(success = true, "Account successfully deleted.")) case None => - NotFound(BasicResult(false, "Account not found")) + NotFound(BasicResult(success = false, "Account not found")) } } yield resp diff --git a/modules/restserver/src/main/scala/sharry/restserver/routes/Conv.scala b/modules/restserver/src/main/scala/sharry/restserver/routes/Conv.scala index 60db43c9..a8ef4b9f 100644 --- a/modules/restserver/src/main/scala/sharry/restserver/routes/Conv.scala +++ b/modules/restserver/src/main/scala/sharry/restserver/routes/Conv.scala @@ -43,40 +43,40 @@ object Conv { def basicResult(ar: AddResult, successMsg: String): BasicResult = ar match { case AddResult.Success => - BasicResult(true, successMsg) + BasicResult(success = true, successMsg) case AddResult.EntityExists(msg) => - BasicResult(false, msg) + BasicResult(success = false, msg) case AddResult.Failure(ex) => - BasicResult(false, ex.getMessage) + BasicResult(success = false, ex.getMessage) } def idResult(successMsg: String)(ar: Either[Throwable, Ident]): IdResult = ar match { - case Right(id) => IdResult(true, successMsg, id) - case Left(ex) => IdResult(false, s"${ex.getClass}: ${ex.getMessage}", Ident.empty) + case Right(id) => IdResult(success = true, successMsg, id) + case Left(ex) => IdResult(success = false, s"${ex.getClass}: ${ex.getMessage}", Ident.empty) } def uploadResult(successMsg: String)(ur: UploadResult[Ident]): IdResult = ur match { case UploadResult.Success(id) => - IdResult(true, successMsg, id) + IdResult(success = true, successMsg, id) case UploadResult.ValidityExceeded(max) => - IdResult(false, s"Maximum validity ($max) exceeded", Ident.empty) + IdResult(success = false, s"Maximum validity ($max) exceeded", Ident.empty) case UploadResult.SizeExceeded(max) => - IdResult(false, s"Maximum size ($max) exceeded", Ident.empty) + IdResult(success = false, s"Maximum size ($max) exceeded", Ident.empty) case UploadResult.PermanentError(msg) => - IdResult(false, msg, Ident.empty) + IdResult(success = false, msg, Ident.empty) } def uploadBasicResult[A](successMsg: String)(ur: UploadResult[A]): BasicResult = ur match { case UploadResult.Success(_) => - BasicResult(true, successMsg) + BasicResult(success = true, successMsg) case UploadResult.ValidityExceeded(max) => - BasicResult(false, s"Maximum validity ($max) exceeded") + BasicResult(success = false, s"Maximum validity ($max) exceeded") case UploadResult.SizeExceeded(max) => - BasicResult(false, s"Maximum size ($max) exceeded") + BasicResult(success = false, s"Maximum size ($max) exceeded") case UploadResult.PermanentError(msg) => - BasicResult(false, msg) + BasicResult(success = false, msg) } } diff --git a/modules/restserver/src/main/scala/sharry/restserver/routes/LoginRoutes.scala b/modules/restserver/src/main/scala/sharry/restserver/routes/LoginRoutes.scala index 6caa8fd9..b99857a3 100644 --- a/modules/restserver/src/main/scala/sharry/restserver/routes/LoginRoutes.scala +++ b/modules/restserver/src/main/scala/sharry/restserver/routes/LoginRoutes.scala @@ -219,7 +219,7 @@ object LoginRoutes { token.account.id, token.account.userLogin, token.account.admin, - true, + success = true, "Login successful", Some(cd.asString), cfg.backend.auth.sessionValid.millis @@ -231,7 +231,15 @@ object LoginRoutes { s"Authentication attempt failure for username $accountName from ip ${req.from.map(_.toInetAddress.getHostAddress).getOrElse("Unknown ip")}" ) *> Ok( - AuthResult(Ident.empty, Ident.empty, false, false, "Login failed.", None, 0L) + AuthResult( + Ident.empty, + Ident.empty, + admin = false, + success = false, + "Login failed.", + None, + 0L + ) ) } } diff --git a/modules/restserver/src/main/scala/sharry/restserver/routes/MailRoutes.scala b/modules/restserver/src/main/scala/sharry/restserver/routes/MailRoutes.scala index 690c8ad6..3c52736c 100644 --- a/modules/restserver/src/main/scala/sharry/restserver/routes/MailRoutes.scala +++ b/modules/restserver/src/main/scala/sharry/restserver/routes/MailRoutes.scala @@ -66,7 +66,10 @@ object MailRoutes { } yield res res.foldF( - err => Ok(BasicResult(false, s"Some recipient addresses are invalid: $err")), + err => + Ok( + BasicResult(success = false, s"Some recipient addresses are invalid: $err") + ), r => Ok(mailSendResult(r)) ) } @@ -74,17 +77,19 @@ object MailRoutes { private def mailSendResult(mr: MailSendResult): BasicResult = mr match { - case MailSendResult.Success => BasicResult(true, "Mail successfully sent.") + case MailSendResult.Success => + BasicResult(success = true, "Mail successfully sent.") case MailSendResult.SendFailure(ex) => - BasicResult(false, s"Mail sending failed: ${ex.getMessage}") - case MailSendResult.NoRecipients => BasicResult(false, "There are no recipients") + BasicResult(success = false, s"Mail sending failed: ${ex.getMessage}") + case MailSendResult.NoRecipients => + BasicResult(success = false, "There are no recipients") case MailSendResult.NoSender => BasicResult( - false, + success = false, "There are no sender addresses specified. You " + "may need to add an e-mail address to your account." ) case MailSendResult.FeatureDisabled => - BasicResult(false, "The mail feature is disabled") + BasicResult(success = false, "The mail feature is disabled") } } diff --git a/modules/restserver/src/main/scala/sharry/restserver/routes/NotifyRoutes.scala b/modules/restserver/src/main/scala/sharry/restserver/routes/NotifyRoutes.scala index bac07082..7b3e17e5 100644 --- a/modules/restserver/src/main/scala/sharry/restserver/routes/NotifyRoutes.scala +++ b/modules/restserver/src/main/scala/sharry/restserver/routes/NotifyRoutes.scala @@ -45,25 +45,25 @@ object NotifyRoutes { private def basicResult(n: List[NotifyResult]): BasicResult = n match { case Nil => - BasicResult(false, "There is no email to notify") + BasicResult(success = false, "There is no email to notify") case NotifyResult.InvalidAlias :: Nil => - BasicResult(false, "Invalid alias") + BasicResult(success = false, "Invalid alias") case NotifyResult.FeatureDisabled :: Nil => - BasicResult(false, "Mail feature is disabled.") + BasicResult(success = false, "Mail feature is disabled.") case _ => val success = n.filter(_.isSuccess).size val fails = n.filter(_.isError).size if (fails > 0 && success == 0) - BasicResult(false, s"Sending failed to all recipients ($fails)") + BasicResult(success = false, s"Sending failed to all recipients ($fails)") else if (fails > 0) BasicResult( - true, + success = true, s"Sending succeeded for $success, but failed for $fails recipients" ) - else BasicResult(true, "Mail sent") + else BasicResult(success = true, "Mail sent") } } diff --git a/modules/restserver/src/main/scala/sharry/restserver/routes/RegisterRoutes.scala b/modules/restserver/src/main/scala/sharry/restserver/routes/RegisterRoutes.scala index e0c7475f..395151fd 100644 --- a/modules/restserver/src/main/scala/sharry/restserver/routes/RegisterRoutes.scala +++ b/modules/restserver/src/main/scala/sharry/restserver/routes/RegisterRoutes.scala @@ -48,25 +48,25 @@ object RegisterRoutes { def convert(r: NewInviteResult): InviteResult = r match { case NewInviteResult.Success(id) => - InviteResult(true, "New invitation created.", Some(id)) + InviteResult(success = true, "New invitation created.", Some(id)) case NewInviteResult.InvitationDisabled => - InviteResult(false, "Signing up is not enabled for invitations.", None) + InviteResult(success = false, "Signing up is not enabled for invitations.", None) case NewInviteResult.PasswordMismatch => - InviteResult(false, "Password is invalid.", None) + InviteResult(success = false, "Password is invalid.", None) } def convert(r: SignupResult): BasicResult = r match { case SignupResult.AccountExists => - BasicResult(false, "An account with this name already exists.") + BasicResult(success = false, "An account with this name already exists.") case SignupResult.InvalidInvitationKey => - BasicResult(false, "Invalid invitation key.") + BasicResult(success = false, "Invalid invitation key.") case SignupResult.SignupClosed => - BasicResult(false, "Sorry, registration is closed.") + BasicResult(success = false, "Sorry, registration is closed.") case SignupResult.Failure(ex) => - BasicResult(false, s"Internal error: ${ex.getMessage}") + BasicResult(success = false, s"Internal error: ${ex.getMessage}") case SignupResult.Success => - BasicResult(true, "Signup successful") + BasicResult(success = true, "Signup successful") } def convert(r: Registration): RegisterData = diff --git a/modules/restserver/src/main/scala/sharry/restserver/webapp/TemplateRoutes.scala b/modules/restserver/src/main/scala/sharry/restserver/webapp/TemplateRoutes.scala index bdca1e70..39a78839 100644 --- a/modules/restserver/src/main/scala/sharry/restserver/webapp/TemplateRoutes.scala +++ b/modules/restserver/src/main/scala/sharry/restserver/webapp/TemplateRoutes.scala @@ -90,7 +90,7 @@ object TemplateRoutes { def loadUrl[F[_]: Sync](url: URL): F[String] = Stream .bracket(Sync[F].blocking(url.openStream))(in => Sync[F].blocking(in.close)) - .flatMap(in => io.readInputStream(in.pure[F], 64 * 1024, false)) + .flatMap(in => io.readInputStream(in.pure[F], 64 * 1024, closeAfterUse = false)) .through(text.utf8.decode) .compile .fold("")(_ + _) diff --git a/modules/store/src/main/scala/sharry/store/ComputeChecksumConfig.scala b/modules/store/src/main/scala/sharry/store/ComputeChecksumConfig.scala index 82f7c6cc..a816609b 100644 --- a/modules/store/src/main/scala/sharry/store/ComputeChecksumConfig.scala +++ b/modules/store/src/main/scala/sharry/store/ComputeChecksumConfig.scala @@ -23,7 +23,7 @@ case class ComputeChecksumConfig( object ComputeChecksumConfig { val defaultParallel: Int = math.min(8, math.max(parallelMin, parallelMax)) - val default = ComputeChecksumConfig(true, 5000, 0, true) + val default = ComputeChecksumConfig(enable = true, 5000, 0, useDefault = true) private def parallelMin: Int = 1 private def parallelMax: Int = Runtime.getRuntime.availableProcessors() - 1 diff --git a/modules/store/src/main/scala/sharry/store/records/RPublishShare.scala b/modules/store/src/main/scala/sharry/store/records/RPublishShare.scala index d9cb0c09..6f691101 100644 --- a/modules/store/src/main/scala/sharry/store/records/RPublishShare.scala +++ b/modules/store/src/main/scala/sharry/store/records/RPublishShare.scala @@ -81,7 +81,7 @@ object RPublishShare { now <- Timestamp.current[ConnectionIO] id <- Ident.randomId[ConnectionIO] validity <- RShare.getDuration(share) - record = RPublishShare(id, share, true, 0, None, now, now.plus(validity), now) + record = RPublishShare(id, share, enabled = true, 0, None, now, now.plus(validity), now) _ <- insert(record) } yield record