diff options
author | Rasmus Dahlberg <rasmus@mullvad.net> | 2022-05-10 21:40:55 +0200 |
---|---|---|
committer | Rasmus Dahlberg <rasmus@mullvad.net> | 2022-05-10 21:40:55 +0200 |
commit | 375786cc95a4117cc33b77fcb6ce40b160f60260 (patch) | |
tree | 5285ee54b7481a12161984a6fc189500072dbcee /archive | |
parent | becf2425af657f425ebab9652130abd6dd565ab8 (diff) |
persist pads from meeting minutes
Diffstat (limited to 'archive')
-rw-r--r-- | archive/2022-05-10--api-error-codes | 103 | ||||
-rw-r--r-- | archive/2022-05-10--availability-monitoring-requirements.md | 38 |
2 files changed, 141 insertions, 0 deletions
diff --git a/archive/2022-05-10--api-error-codes b/archive/2022-05-10--api-error-codes new file mode 100644 index 0000000..d80455f --- /dev/null +++ b/archive/2022-05-10--api-error-codes @@ -0,0 +1,103 @@ +Notes from in-person meet to make error-cases in api.md more well-defined. + +Status codes 200, 404, 405, and 500 are not endpoint specific. When we take a +pass over api.md, it might be a good idea to not list them redundantly for each +endpoint. + +§3.1, get-tree-head-to-cosign + * http method - Get + * Input - None + * Output - timestamp, tree_size, root_hash, signature + * Users - Witness + * Status Code + * 200 - Success + * 404 - Not Found + * 405 - Method Not Allowed + * 500 - Internal Server Error + +§3.2, get-tree-head-cosigned + * http method - Get + * Input - None + * Output - timestamp, tree_size, root_hash, signature, cosignature, key_hash + * Users - signers, monitors + * Status Code + * 200 - Success + * consider request successful even if the list of cosignatures is empty + * update api.md to allow empty list by simply not listing a key-value pair + * 404 - Not Found + * 405 - Method Not Allowed + * 500 - Internal Server Error + +§3.3 - get-inclusion-proof + * http method - Get + * Input - tree_size, leaf_hash + * Output - leaf_index, inclusion_path + * Users - signer, monitor + * Status Code + * 200 - Success + * 400 - Bad Request (bad input) + * 404 - Not Found + * 405 - Method Not Allowed + * 500 - Internal Server Error + +§3.4, get-consistency-proof + * http method - Get + * Input - old_size, new_size + * Output - consistency_path + * Users - witness, optionally signer and monitor + * Status Code + * 200 - Success + * 400 - Bad Request (bad input) + * 404 - Not Found + * 405 - Method Not Allowed + * 500 - Internal Server Error + +§3.5, get-leaves + * http method - Get + * Input - start_size, end_size + * Output - shard_hint, checksum, signature, key_hash + * Users - monitors + * Status Code + * 200 - Success + * 400 - Bad Request (bad input) + * 404 - Not Found + * 405 - Method Not Allowed + * 500 - Internal Server Error + +§3.6, add-leaf + * http method - Post + * Input - shard_hint, message, signature, public_key, domain_hint + * Output - None + * Users - Signer + * Status Code + * 200 - Success (will be in the log after signing next tree head) + * 202 - Accepted (trying to add to the log, still not commited) + * 400 - Bad Request (e.g., invalid signature, TBD: duplicate as well?) + * 403 - Forbidden (bad domain hint) + * 404 - Not Found + * 405 - Method Not Allowed + * 429 - Too Many Requests (rate-limit kicked in for domain hint) + * 500 - Internal Server Error + +§3.7, add-cosignature + * http method - Post + * Input - cosignature, key_hash + * Output - None + * Users - Witness + * Status Code + * 200 - Success + * 400 - Bad Request (e.g., bad signature) + * 403 - Forbidden (witness key_hash is not configured) + * 404 - Not Found + * 405 - Method Not Allowed + * 500 - Internal Server Error + +Note that clients may encountered other status codes from components that are +placed in front of log-go, e.g., from nginx/apache/etc: + * 301 - Moved Permanently + * 307 - Temporary Redirect + * 429 - Too Many Requests + * 503 - Service Unavailable + +Human-readable string should be free-form and not "error=" due to the above. +I.e., nginx is not going to give a text string on the format "error=". diff --git a/archive/2022-05-10--availability-monitoring-requirements.md b/archive/2022-05-10--availability-monitoring-requirements.md new file mode 100644 index 0000000..7522503 --- /dev/null +++ b/archive/2022-05-10--availability-monitoring-requirements.md @@ -0,0 +1,38 @@ +# Requirements for availability monitoring + + - System requirements + - Runs on a modern linux distro + - System can send email + - System can run sigsum-debug binary + - Root access is not required + - Leveled alerts, aka severity + - Error + - Warning + - Alerts via email + - Check endpoint reachability + - API §3.1 - §3.7 + - Enumerate the different ways to use an endpoint + - http, https + - ipv4, ipv6 + - onion +- Verify response data (the output of Sigsum endpoints) + - E.g., timestamp or number of witness signatures on get-tree-head-to-cosign +- Perform checks from multiple vantage points + +# Concrete milestones + +220515 + + - Sigsum endpoints + - Either IPv4 or IPv6 (whatever the system's curl uses) + - Only HTTPS + - When do they 2XX? + - No alert state + - One alert per failure + - Configurable test frequency (~1h hour) + - Notifications by email only to one single address + - "Be done very quickly but also very imperfectly" + +220530 +220831 +221031 |