libeufin

Integration and sandbox testing for FinTech APIs and data formats
Log | Files | Refs | Submodules | README | LICENSE

commit 63e7e02ca0e7b31ac90fa247dd2df79d9a493c03
parent f5e670cc0bb8dc8424eb1aa891fc0161064395ca
Author: MS <ms@taler.net>
Date:   Mon, 25 Sep 2023 14:06:49 +0200

Fixing comments.

Removing accidental "//" from single-line comments
and placing a FIXME above the spaCaptchaURL field.

Diffstat:
Mbank/src/main/kotlin/tech/libeufin/bank/CorebankApiHandlers.kt | 6+++---
Mbank/src/main/kotlin/tech/libeufin/bank/Main.kt | 2+-
2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/bank/src/main/kotlin/tech/libeufin/bank/CorebankApiHandlers.kt b/bank/src/main/kotlin/tech/libeufin/bank/CorebankApiHandlers.kt @@ -145,7 +145,7 @@ fun Routing.accountsMgmtHandlers(db: Database, ctx: BankApplicationContext) { passwordHash = CryptoUtil.hashpw(req.password), ) val newCustomerRowId = db.customerCreate(newCustomer) - ?: throw internalServerError("New customer INSERT failed despite the previous checks") // Crashing here won't break data consistency between customers // and bank accounts, because of the idempotency. Client will // just have to retry. + ?: throw internalServerError("New customer INSERT failed despite the previous checks") // Crashing here won't break data consistency between customers and bank accounts, because of the idempotency. Client will just have to retry. val maxDebt = ctx.defaultCustomerDebtLimit val newBankAccount = BankAccount( hasDebt = false, @@ -192,7 +192,7 @@ fun Routing.accountsMgmtHandlers(db: Database, ctx: BankApplicationContext) { val c = call.authenticateBankRequest(db, TokenScope.readonly) ?: throw unauthorized("Login failed") val resourceName = call.maybeUriComponent("USERNAME") ?: throw badRequest( hint = "No username found in the URI", talerErrorCode = TalerErrorCode.TALER_EC_GENERIC_PARAMETER_MISSING - ) // Checking resource name only if Basic auth was used. // Successful tokens do not need this check, they just pass. + ) // Checking resource name only if Basic auth was used. Successful tokens do not need this check, they just pass. if (((c.login != resourceName) && (c.login != "admin")) && (call.getAuthToken() == null)) throw forbidden("No rights on the resource.") val customerData = db.customerGetFromLogin(c.login) ?: throw internalServerError("Customer '${c.login} despite being authenticated.'") @@ -370,7 +370,7 @@ fun Routing.accountsMgmtHandlers(db: Database, ctx: BankApplicationContext) { val paytoWithoutParams = stripIbanPayto(txData.payto_uri) val subject = payto.message ?: throw badRequest("Wire transfer lacks subject") val debtorId = c.dbRowId - ?: throw internalServerError("Debtor database ID not found") // This performs already a SELECT on the bank account, // like the wire transfer will do as well later! + ?: throw internalServerError("Debtor database ID not found") // This performs already a SELECT on the bank account, like the wire transfer will do as well later! logger.info("creditor payto: $paytoWithoutParams") val creditorCustomerData = db.bankAccountGetFromInternalPayto(paytoWithoutParams) ?: throw notFound( "Creditor account not found", TalerErrorCode.TALER_EC_END // FIXME: define this EC. diff --git a/bank/src/main/kotlin/tech/libeufin/bank/Main.kt b/bank/src/main/kotlin/tech/libeufin/bank/Main.kt @@ -105,7 +105,7 @@ data class BankApplicationContext( */ val maxAuthTokenDurationUs: Long, /** - * Max token duration in microseconds. + * FIXME/TBD. */ val spaCaptchaURL: String?, )