mirror of
https://github.com/OMGeeky/tarpc.git
synced 2025-12-28 15:22:30 +01:00
905e5be8bb
# New Crates - crate rpc contains the core client/server request-response framework, as well as a transport trait. - crate bincode-transport implements a transport that works almost exactly as tarpc works today (not to say it's wire-compatible). - crate trace has some foundational types for tracing. This isn't really fleshed out yet, but it's useful for in-process log tracing, at least. All crates are now at the top level. e.g. tarpc-plugins is now tarpc/plugins rather than tarpc/src/plugins. tarpc itself is now a *very* small code surface, as most functionality has been moved into the other more granular crates. # New Features - deadlines: all requests specify a deadline, and a server will stop processing a response when past its deadline. - client cancellation propagation: when a client drops a request, the client sends a message to the server informing it to cancel its response. This means cancellations can propagate across multiple server hops. - trace context stuff as mentioned above - more server configuration for total connection limits, per-connection request limits, etc. # Removals - no more shutdown handle. I left it out for now because of time and not being sure what the right solution is. - all async now, no blocking stub or server interface. This helps with maintainability, and async/await makes async code much more usable. The service trait is thusly renamed Service, and the client is renamed Client. - no built-in transport. Tarpc is now transport agnostic (see bincode-transport for transitioning existing uses). - going along with the previous bullet, no preferred transport means no TLS support at this time. We could make a tls transport or make bincode-transport compatible with TLS. - a lot of examples were removed because I couldn't keep up with maintaining all of them. Hopefully the ones I kept are still illustrative. - no more plugins! # Open Questions 1. Should client.send() return `Future<Response>` or `Future<Future<Response>>`? The former appears more ergonomic but it doesn’t allow concurrent requests with a single client handle. The latter is less ergonomic but yields back control of the client once it’s successfully sent out the request. Should we offer fns for both? 2. Should rpc service! Fns take &mut self or &self or self? The service needs to impl Clone anyway, technically we only need to clone it once per connection, and then leave it up to the user to decide if they want to clone it per RPC. In practice, everyone doing nontrivial stuff will need to clone it per RPC, I think. 3. Do the request/response structs look ok? 4. Is supporting server shutdown/lameduck important? Fixes #178 #155 #124 #104 #83 #38
99 lines
2.6 KiB
Bash
Executable File
99 lines
2.6 KiB
Bash
Executable File
#!/bin/bash
|
|
|
|
# Copyright 2016 Google Inc. All Rights Reserved.
|
|
#
|
|
# Licensed under the MIT License, <LICENSE or http://opensource.org/licenses/MIT>.
|
|
# This file may not be copied, modified, or distributed except according to those terms.
|
|
|
|
# Pre-push hook for the tarpc repository. To use this hook, copy it to .git/hooks in your repository
|
|
# root.
|
|
#
|
|
# This hook runs tests to make sure only working code is being pushed. If present, rustup is used
|
|
# to build and test the code on the appropriate toolchains. The working copy must not contain
|
|
# uncommitted changes, since the script currently just runs cargo build/test in the working copy.
|
|
#
|
|
# Options:
|
|
#
|
|
# - TARPC_ALLOW_DIRTY, default = 0
|
|
#
|
|
# Setting this variable to 1 will run tests even though there are code changes in the working
|
|
# copy. Set to 0 by default, since the intent is to test the code that's being pushed, not changes
|
|
# still in the working copy.
|
|
#
|
|
# Note that these options are most useful for testing the hooks themselves. Use git push --no-verify
|
|
# to skip the pre-push hook altogether.
|
|
|
|
RED='\033[0;31m'
|
|
GREEN='\033[0;32m'
|
|
YELLOW='\033[0;33m'
|
|
NC='\033[0m' # No Color
|
|
|
|
PREFIX="${GREEN}[PREPUSH]${NC}"
|
|
FAILURE="${RED}FAILED${NC}"
|
|
WARNING="${YELLOW}[WARNING]${NC}"
|
|
SKIPPED="${YELLOW}SKIPPED${NC}"
|
|
SUCCESS="${GREEN}ok${NC}"
|
|
|
|
printf "${PREFIX} Clean working copy ... "
|
|
git diff --exit-code &>/dev/null
|
|
if [ "$?" == 0 ]; then
|
|
printf "${SUCCESS}\n"
|
|
else
|
|
if [ "${TARPC_ALLOW_DIRTY}" == "1" ]
|
|
then
|
|
printf "${SKIPPED}\n"
|
|
else
|
|
printf "${FAILURE}\n"
|
|
exit 1
|
|
fi
|
|
fi
|
|
|
|
PREPUSH_RESULT=0
|
|
|
|
try_run() {
|
|
TEXT=$1
|
|
shift
|
|
printf "${PREFIX} ${TEXT}"
|
|
OUTPUT=$($@ 2>&1)
|
|
if [ "$?" != "0" ]; then
|
|
printf "${FAILURE}, output shown below\n"
|
|
printf "\n\n"
|
|
printf "$OUTPUT"
|
|
printf "\n\n"
|
|
PREPUSH_RESULT=1
|
|
return 1
|
|
else
|
|
printf "${SUCCESS}\n"
|
|
fi
|
|
}
|
|
|
|
TOOLCHAIN_RESULT=0
|
|
check_toolchain() {
|
|
printf "${PREFIX} Checking for $1 toolchain ... "
|
|
if [[ $(rustup toolchain list) =~ $1 ]]; then
|
|
printf "${SUCCESS}\n"
|
|
else
|
|
TOOLCHAIN_RESULT=1
|
|
PREPUSH_RESULT=1
|
|
printf "${FAILURE}\n"
|
|
fi
|
|
}
|
|
|
|
printf "${PREFIX} Checking for rustup or current toolchain directive... "
|
|
command -v rustup &>/dev/null
|
|
if [ "$?" == 0 ]; then
|
|
printf "${SUCCESS}\n"
|
|
|
|
check_toolchain nightly
|
|
if [ ${TOOLCHAIN_RESULT} == 1 ]; then
|
|
exit 1
|
|
fi
|
|
|
|
try_run "Building ... " cargo build --color=always
|
|
try_run "Testing ... " cargo test --color=always
|
|
try_run "Doc Test ... " cargo clean && cargo build --tests && rustdoc --test README.md --edition 2018 -L target/debug/deps -Z unstable-options
|
|
|
|
fi
|
|
|
|
exit $PREPUSH_RESULT
|