Cargo.toml: Update all versions to 1.0.0

Commit made automatically by update-versions.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

changelog: Document that this is version 1.0.0

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

changelog: Document changes since 0.7.3

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

daemon: Move src_service into services! call

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

inventory: Postpone this TODO

So far things still seem to be working, despite upstream rust problems
with #[used].

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Document new build disk space requirement - reduced !

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

daemon: Reorder routes in order of likely frequency

I'm not sure if this is at all relevant but it seems sensible
and is at worst harmless.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

actix: Revamp /_/{resource} handling

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

release checklist: Drop entry about nightly

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Formulaic doc comments for other crates

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Bespoke doc comments for otter, otter-cli and otter-daemon

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Drop unused dependency: failure

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

cargo: Update zip (zipfile) dependency

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Cargo update

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Makefile: Build wasm-bindgen with selected rust version too

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Deploy release build

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Turn on integer overflow checks for release builds

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

actix: Fix CORS

The docs aren't entirely clear but suggest that the default allowed
origins is All.  But dbg! output seems to contradict this.  And we
should support more headers, and say that we don't vary our CORS at
all.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Docs update: Not beta, works on stable

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Makefile: Arrangements for honouring RUST_VERSION

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Update TODOs

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

apitest: Fix a write() call that should be write_all()

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Update TODOs

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Use Educe to impl Default on a couple of types in clisupport

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

parse_slice_hex: Fix to properly handle odd-length slices

Previously it would accept some of them, and parse them in a weird
way.  I don't think this is actually important.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

PlayerUpdates: Rename update log starting functions and types

Prompted by clippy complaining about the old weird new() function.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

termprogress: Rename reporter() functions returning Box<dyn>

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

packetframe: Fix a write() call that should be write_all()

I think this was probably a bug.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Suppress a buggy lint

This fires on process_all_players_for_account, but the collect
*is* needed and the suggestion doesn't compile.

I wasn't able to make a minimal repro, but there is what looks like a
report of the same thing (and many other issues with this lint).

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

RawToken: Replace constructor with a From impl

No functional change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Prefer .len() > 0 to ! .is_empty()

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

InstanceBundles: impl Default and use that for new()

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

asseturl: Fix a wrong call to write, to be write_all

In practice this bug is latent, buecause DigestWrite will take the
whole thing.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes, and allow lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

zcoord: Move from_str impl for Mutable to be FromStr impl

Suggested by clippy.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

zcoord: Move from_str impl to be (only) in FromStr impl

Suggested by clippy.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

zcoord: Remove redundant to_string method

This is the same as the Display impl.  Suggested by clippy.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: allow some lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Miscellaneous minor changes

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Html type: impl Default and derive new()

Default suggested by clippy.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Makefile: Support for controlling the Rust version

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: allow some lints

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: wip machinery, and transfer the one allowed lint so far

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

actix: Drop a leftover dbg

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Drop remaining feature enablements.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Remove one use of TAIT

This is our remaining usntable feature.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Drop IteratorExt extension trait

We used this once.  Not a successful experiment I think.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

actix: Document the situation with HEAD etc.

This is not very good really, but I guess it will do for now.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

daemon: Remove a stray commented-out import thing.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

actix Files: Work around content-type breakage

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

actix Files: Do our content-type mangling first

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

actix Files: Break out FILES_PATH

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

actix Files: change error handling for middleware fixup fn

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

actix files content-type fixup: wip, call fn middlware

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

daemon: Fix whitespace error

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Abolish JsonLen and json_len

This was for SSE buffer management, but we don't need it now.
And it was quite fiddly!

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

SSE: Adjust error handling somewhat

Have UpdateReader::read throw a bespoke error.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

SSE: Redo buffer handling

Now we just write into a Vec.  This is rather simpler.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Refine a TODO

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

server: Support explicit specification of addresses to listen on

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

server: Move listening address config resolution to config

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

server: Listen on localhost only by default, regardless of debug

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

server: Listen on port 8000 by default, regardless of debug

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Note a TODO

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

actix routing: Use a conventional prefix for route "function" names

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

resource leaf routing: Document why we're leaving it order-sensitive

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Revert "resource leaf routing: Attempt via impl FromRequest"

This reverts commit 383fab0156489a705901ba06580c60215dc313c6.

resource leaf routing: Attempt via impl FromRequest

I wrote this thinking that implementing FromRequest, so that we can
return Status::NOT_FOUND would work to pass on to the next route.
But, no.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Tidying: Simplify a unit error struct

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

actix cleanup: import futures::future

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

actix cleanup: Minor reformattint

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Cargo.nail: Drop our own directory

We aren't nailing anything right now.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

web templates: Use stem for template names (without .tera extension)

This moves a bit of code to startup, and makes the macro use a bit
cleaner again.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

web templates: Provide our own directory scanner

We don't know exactly what Tera::new() does, but in any case we want
to do something it doesn't support: have template names which do not
contain the .tera file extension.

No functional change in this commit.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Rocket cleanup: Call our tera tera

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>