[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[taler-exchange] branch master updated: implement rh caching
|
From: |
gnunet |
|
Subject: |
[taler-exchange] branch master updated: implement rh caching |
|
Date: |
Fri, 17 Jan 2020 12:52:35 +0100 |
This is an automated email from the git hooks/post-receive script.
grothoff pushed a commit to branch master
in repository exchange.
The following commit(s) were added to refs/heads/master by this push:
new 540b22ce implement rh caching
540b22ce is described below
commit 540b22ce1ca41e66574eb156678a7ff288403951
Author: Christian Grothoff <address@hidden>
AuthorDate: Fri Jan 17 12:52:24 2020 +0100
implement rh caching
---
src/exchange/taler-exchange-httpd.c | 28 ++++-
src/exchange/taler-exchange-httpd_refund.c | 169 ++++++++++++++++------------
src/exchangedb/plugin_exchangedb_postgres.c | 49 ++++++++
src/include/taler_exchangedb_plugin.h | 17 +++
4 files changed, 185 insertions(+), 78 deletions(-)
diff --git a/src/exchange/taler-exchange-httpd.c
b/src/exchange/taler-exchange-httpd.c
index 1b9333df..44ac82d6 100644
--- a/src/exchange/taler-exchange-httpd.c
+++ b/src/exchange/taler-exchange-httpd.c
@@ -66,6 +66,11 @@ struct ExchangeHttpRequestClosure
* Opaque parsing context.
*/
void *opaque_post_parsing_context;
+
+ /**
+ * Cached request handler for this request (once we have found one).
+ */
+ struct TEH_RequestHandler *rh;
};
@@ -360,7 +365,6 @@ handle_mhd_request (void *cls,
"<html><title>404: not found</title></html>", 0,
&TEH_MHD_handler_static_response, MHD_HTTP_NOT_FOUND
};
- struct TEH_RequestHandler *rh;
struct ExchangeHttpRequestClosure *ecls = *con_cls;
int ret;
void **inner_cls;
@@ -413,18 +417,28 @@ handle_mhd_request (void *cls,
"Handling request (%s) for URL '%s'\n",
method,
url);
-
+ /* on repeated requests, check our cache first */
+ if (NULL != ecls->rh)
+ {
+ ret = ecls->rh->handler (ecls->rh,
+ connection,
+ inner_cls,
+ upload_data,
+ upload_data_size);
+ GNUNET_async_scope_restore (&old_scope);
+ return ret;
+ }
if (0 == strcasecmp (method,
MHD_HTTP_METHOD_HEAD))
method = MHD_HTTP_METHOD_GET; /* treat HEAD as GET here, MHD will do the
rest */
for (unsigned int i = 0; NULL != handlers[i].url; i++)
{
- rh = &handlers[i];
+ struct TEH_RequestHandler *rh = &handlers[i];
+
if (0 != strcmp (url, rh->url))
continue;
/* The URL is a match! What we now do depends on the method. */
-
if (0 == strcasecmp (method, MHD_HTTP_METHOD_OPTIONS))
{
GNUNET_async_scope_restore (&old_scope);
@@ -435,8 +449,9 @@ handle_mhd_request (void *cls,
(0 == strcasecmp (method,
rh->method)) )
{
- /* FIXME: consider caching 'rh' in '**connection_cls' to
- avoid repeated lookup! */
+ /* cache to avoid the loop next time */
+ ecls->rh = rh;
+ /* run handler */
ret = rh->handler (rh,
connection,
inner_cls,
@@ -446,6 +461,7 @@ handle_mhd_request (void *cls,
return ret;
}
}
+ /* No handler matches, generate not found */
ret = TEH_MHD_handler_static_response (&h404,
connection,
inner_cls,
diff --git a/src/exchange/taler-exchange-httpd_refund.c
b/src/exchange/taler-exchange-httpd_refund.c
index cb0a0f15..4effdb2c 100644
--- a/src/exchange/taler-exchange-httpd_refund.c
+++ b/src/exchange/taler-exchange-httpd_refund.c
@@ -1,6 +1,6 @@
/*
This file is part of TALER
- Copyright (C) 2014-2017 Inria and GNUnet e.V.
+ Copyright (C) 2014-2020 Taler Systems SA
TALER is free software; you can redistribute it and/or modify it under the
terms of the GNU Affero General Public License as published by the Free
Software
@@ -131,6 +131,24 @@ reply_refund_conflict (struct MHD_Connection *connection,
}
+/**
+ * Closure for the transaction.
+ */
+struct TALER_EXCHANGEDB_RefundContext
+{
+ /**
+ * Information about the refund.
+ */
+ const struct TALER_EXCHANGEDB_Refund *refund;
+
+ /**
+ * Expected refund fee by the denomination of the coin.
+ */
+ struct TALER_Amount expect_fee;
+
+};
+
+
/**
* Execute a "/refund" transaction. Returns a confirmation that the
* refund was successful, or a failure if we are not aware of a
@@ -155,20 +173,15 @@ refund_transaction (void *cls,
struct TALER_EXCHANGEDB_Session *session,
int *mhd_ret)
{
- const struct TALER_EXCHANGEDB_Refund *refund = cls;
+ struct TALER_EXCHANGEDB_RefundContext *rc = cls;
+ const struct TALER_EXCHANGEDB_Refund *refund = rc->refund;
struct TALER_EXCHANGEDB_TransactionList *tl;
const struct TALER_EXCHANGEDB_DepositListEntry *dep;
const struct TALER_EXCHANGEDB_RefundListEntry *ref;
- struct TEH_KS_StateHandle *mks;
- struct TALER_EXCHANGEDB_DenominationKeyIssueInformation *dki;
- struct TALER_Amount expect_fee;
enum GNUNET_DB_QueryStatus qs;
int deposit_found;
int refund_found;
int fee_cmp;
- unsigned int hc;
- enum TALER_ErrorCode ec;
- struct TALER_CoinPublicInfo coin_info;
dep = NULL;
ref = NULL;
@@ -356,64 +369,9 @@ refund_transaction (void *cls,
TALER_EC_REFUND_INSUFFICIENT_FUNDS);
return GNUNET_DB_STATUS_HARD_ERROR;
}
- qs = TEH_plugin->get_known_coin (TEH_plugin->cls,
- session,
- &refund->coin.coin_pub,
- &coin_info);
- if (0 > qs)
- {
- TEH_plugin->free_coin_transaction_list (TEH_plugin->cls,
- tl);
- if (GNUNET_DB_STATUS_HARD_ERROR == qs)
- {
- GNUNET_break (0); /* should be impossible by foreign key constraint! */
- *mhd_ret = reply_refund_failure (connection,
- MHD_HTTP_NOT_FOUND,
- TALER_EC_REFUND_COIN_NOT_FOUND);
- }
- return qs;
- }
- GNUNET_CRYPTO_rsa_signature_free (coin_info.denom_sig.rsa_signature);
- coin_info.denom_sig.rsa_signature = NULL; /* just to be safe */
- // FIXME: do this outside of transaction function?
/* Check refund fee matches fee of denomination key! */
- mks = TEH_KS_acquire (GNUNET_TIME_absolute_get ());
- if (NULL == mks)
- {
- TALER_LOG_ERROR ("Lacking keys to operate\n");
- TEH_plugin->free_coin_transaction_list (TEH_plugin->cls,
- tl);
- *mhd_ret = TALER_MHD_reply_with_error (connection,
- MHD_HTTP_INTERNAL_SERVER_ERROR,
- TALER_EC_EXCHANGE_BAD_CONFIGURATION,
- "no keys");
- return GNUNET_DB_STATUS_HARD_ERROR;
- }
- dki = TEH_KS_denomination_key_lookup_by_hash (mks,
- &coin_info.denom_pub_hash,
- TEH_KS_DKU_DEPOSIT,
- &ec,
- &hc);
- if (NULL == dki)
- {
- /* DKI not found, but we do have a coin with this DK in our database;
- not good... */
- GNUNET_break (0);
- TEH_KS_release (mks);
- TEH_plugin->free_coin_transaction_list (TEH_plugin->cls,
- tl);
- *mhd_ret = TALER_MHD_reply_with_error (connection,
- hc,
- ec,
- "denomination not found, but coin
known");
- return GNUNET_DB_STATUS_HARD_ERROR;
- }
- TALER_amount_ntoh (&expect_fee,
- &dki->issue.properties.fee_refund);
fee_cmp = TALER_amount_cmp (&refund->details.refund_fee,
- &expect_fee);
- TEH_KS_release (mks);
-
+ &rc->expect_fee);
if (-1 == fee_cmp)
{
TEH_plugin->free_coin_transaction_list (TEH_plugin->cls,
@@ -464,8 +422,9 @@ static int
verify_and_execute_refund (struct MHD_Connection *connection,
const struct TALER_EXCHANGEDB_Refund *refund)
{
+ struct TALER_EXCHANGEDB_RefundContext rc;
struct TALER_RefundRequestPS rr;
- int mhd_ret;
+ struct GNUNET_HashCode denom_hash;
if (GNUNET_YES !=
TALER_amount_cmp_currency (&refund->details.refund_amount,
@@ -508,13 +467,79 @@ verify_and_execute_refund (struct MHD_Connection
*connection,
TALER_EC_REFUND_MERCHANT_SIGNATURE_INVALID,
"merchant_sig");
}
- if (GNUNET_OK !=
- TEH_DB_run_transaction (connection,
- "run refund",
- &mhd_ret,
- &refund_transaction,
- (void *) refund))
- return mhd_ret;
+
+ /* Fetch the coin's denomination (hash) */
+ {
+ enum GNUNET_DB_QueryStatus qs;
+
+ qs = TEH_plugin->get_coin_denomination (TEH_plugin->cls,
+ NULL,
+ &refund->coin.coin_pub,
+ &denom_hash);
+ if (0 > qs)
+ {
+ GNUNET_break (GNUNET_DB_STATUS_HARD_ERROR == qs);
+ return reply_refund_failure (connection,
+ MHD_HTTP_NOT_FOUND,
+ TALER_EC_REFUND_COIN_NOT_FOUND);
+ }
+ }
+
+ {
+ struct TEH_KS_StateHandle *mks;
+
+ mks = TEH_KS_acquire (GNUNET_TIME_absolute_get ());
+ if (NULL == mks)
+ {
+ TALER_LOG_ERROR ("Lacking keys to operate\n");
+ return TALER_MHD_reply_with_error (connection,
+ MHD_HTTP_INTERNAL_SERVER_ERROR,
+ TALER_EC_EXCHANGE_BAD_CONFIGURATION,
+ "no keys");
+ }
+ /* Obtain information about the coin's denomination! */
+ {
+ struct TALER_EXCHANGEDB_DenominationKeyIssueInformation *dki;
+ unsigned int hc;
+ enum TALER_ErrorCode ec;
+
+ dki = TEH_KS_denomination_key_lookup_by_hash (mks,
+ &denom_hash,
+ TEH_KS_DKU_DEPOSIT,
+ &ec,
+ &hc);
+ if (NULL == dki)
+ {
+ /* DKI not found, but we do have a coin with this DK in our database;
+ not good... */
+ GNUNET_break (0);
+ TEH_KS_release (mks);
+ return TALER_MHD_reply_with_error (connection,
+ hc,
+ ec,
+ "denomination not found, but coin
known");
+ }
+ TALER_amount_ntoh (&rc.expect_fee,
+ &dki->issue.properties.fee_refund);
+ }
+ TEH_KS_release (mks);
+ }
+
+ /* Finally run the actual transaction logic */
+ {
+ int mhd_ret;
+
+ rc.refund = refund;
+ if (GNUNET_OK !=
+ TEH_DB_run_transaction (connection,
+ "run refund",
+ &mhd_ret,
+ &refund_transaction,
+ &rc))
+ {
+ return mhd_ret;
+ }
+ }
return reply_refund_success (connection,
&refund->coin.coin_pub,
&refund->details);
diff --git a/src/exchangedb/plugin_exchangedb_postgres.c
b/src/exchangedb/plugin_exchangedb_postgres.c
index 80f91785..925f2853 100644
--- a/src/exchangedb/plugin_exchangedb_postgres.c
+++ b/src/exchangedb/plugin_exchangedb_postgres.c
@@ -550,6 +550,15 @@ postgres_get_session (void *cls)
" WHERE coin_pub=$1"
" FOR UPDATE;",
1),
+ /* Used in #postgres_get_coin_denomination() to fetch
+ the denomination public key hash for
+ a coin known to the exchange. */
+ GNUNET_PQ_make_prepare ("get_coin_denomination",
+ "SELECT"
+ " denom_pub_hash"
+ " FROM known_coins"
+ " WHERE coin_pub=$1",
+ 1),
/* Lock deposit table; NOTE: we may want to eventually shard the
deposit table to avoid this lock being the main point of
contention limiting transaction performance. */
@@ -3007,6 +3016,45 @@ postgres_get_known_coin (void *cls,
}
+/**
+ * Retrieve the denomination of a known coin.
+ *
+ * @param cls the plugin closure
+ * @param session the database session handle
+ * @param coin_pub the public key of the coin to search for
+ * @param denom_hash[out] where to store the hash of the coins denomination
+ * @return transaction status code
+ */
+static enum GNUNET_DB_QueryStatus
+postgres_get_coin_denomination (void *cls,
+ struct TALER_EXCHANGEDB_Session *session,
+ const struct
+ TALER_CoinSpendPublicKeyP *coin_pub,
+ struct GNUNET_HashCode *denom_hash)
+{
+ struct PostgresClosure *pc = cls;
+ struct GNUNET_PQ_QueryParam params[] = {
+ GNUNET_PQ_query_param_auto_from_type (coin_pub),
+ GNUNET_PQ_query_param_end
+ };
+ struct GNUNET_PQ_ResultSpec rs[] = {
+ GNUNET_PQ_result_spec_auto_from_type ("denom_pub_hash",
+ denom_hash),
+ GNUNET_PQ_result_spec_end
+ };
+
+ GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+ "Getting coin denomination of coin %s\n",
+ TALER_B2S (coin_pub));
+ if (NULL == session)
+ session = postgres_get_session (pc);
+ return GNUNET_PQ_eval_prepared_singleton_select (session->conn,
+ "get_coin_denomination",
+ params,
+ rs);
+}
+
+
/**
* Insert a coin we know of into the DB. The coin can then be
* referenced by tables for deposits, refresh and refund
@@ -7263,6 +7311,7 @@ libtaler_plugin_exchangedb_postgres_init (void *cls)
plugin->count_known_coins = &postgres_count_known_coins;
plugin->ensure_coin_known = &postgres_ensure_coin_known;
plugin->get_known_coin = &postgres_get_known_coin;
+ plugin->get_coin_denomination = &postgres_get_coin_denomination;
plugin->have_deposit = &postgres_have_deposit;
plugin->mark_deposit_tiny = &postgres_mark_deposit_tiny;
plugin->test_deposit_done = &postgres_test_deposit_done;
diff --git a/src/include/taler_exchangedb_plugin.h
b/src/include/taler_exchangedb_plugin.h
index 3178209d..633640e0 100644
--- a/src/include/taler_exchangedb_plugin.h
+++ b/src/include/taler_exchangedb_plugin.h
@@ -1884,6 +1884,23 @@ struct TALER_EXCHANGEDB_Plugin
const struct TALER_CoinSpendPublicKeyP *coin_pub,
struct TALER_CoinPublicInfo *coin_info);
+
+ /**
+ * Retrieve the denomination of a known coin.
+ *
+ * @param cls the plugin closure
+ * @param session the database session handle
+ * @param coin_pub the public key of the coin to search for
+ * @param denom_hash[out] where to store the hash of the coins denomination
+ * @return transaction status code
+ */
+ enum GNUNET_DB_QueryStatus
+ (*get_coin_denomination)(void *cls,
+ struct TALER_EXCHANGEDB_Session *session,
+ const struct TALER_CoinSpendPublicKeyP *coin_pub,
+ struct GNUNET_HashCode *denom_hash);
+
+
/**
* Check if we have the specified deposit already in the database.
*
--
To stop receiving notification emails like this one, please contact
address@hidden.
| [Prev in Thread] |
Current Thread |
[Next in Thread] |
- [taler-exchange] branch master updated: implement rh caching,
gnunet <=