fix hang at end of PUT to proxied p2p http remote

sendExactly will now be sure to evaluate the whole lazy ByteString.

In this case, the lazy ByteString was exactly the right lenth.
But, it seems that L.take caused it to not actually be fully evaluated.

In servePut, this manifested as gather never being fully evaluated,
which caused the hang.

Very, very subtle, and horrible bug. Clearly the use of lazy ByteString
(or really just laziness) is at fault, and it would be very worth moving
to conduit or whatever to avoid this.

update

implement proxy connection pool

removeOldestProxyConnectionPool will be innefficient the larger the pool
is. A better data structure could be more efficient. Eg, make each value
in the pool include the timestamp of its oldest element, then the oldest
value can be found and modified, rather than rebuilding the whole Map.

But, for pools of a few hundred items, this should be fine. It's O(n*n log n)
or so.

Also, when more than 1 connection with the same pool key exists,
it's efficient even for larger pools, since removeOldestProxyConnectionPool
is not needed.

The default of 1 idle connection could perhaps be larger.. like the
number of jobs? Otoh, it seems good to ramp up and down the number of
connections, which does happen. With 1, there is at most one stale
connection, which might cause a request to fail.

closeP2PConnection on interrupted GET

clean up after http p2p proxy GET is interrupted

There was an annex worker thread that did not get stopped.

It was stuck in ReceiveMessage from the P2PHandleTMVar.

Fixed by making P2PHandleTMVar closeable.

In serveGet, releaseP2PConnection has to come first, else the
annexworker may not shut down, if it's waiting to read from it.

In proxyConnection, call closeRemoteSide in order to wait for the ssh
process (for example).

close proxy remote side when done with it

oops

oops

clean up protocol version for proxying

The proxy always checks the protocol version of a remote before talking
to it in a version-specific way, so the protocol version in the ProxyParams
is the client's protocol version. The remote will always be at the same or
an older protocol version than the client.

Note that in relayDATAFinish, when the client is at protocol version 0,
the remote must thus be as well, and that's why its version is not
checked in the case for that.

With that clarified, it's evident that, in P2P.Http.State, there's no
need to look at the proxied remote's protocol version at all.

fix setting protocol version for v0 server

Before it was leaving the protocol version at the default, when it
was actually v0.

v0 has only ever been used over tor, which is probably why this bug was
never noticed.

fix hang in GET from http p2p proxy

serverP2PConnection = proxyfromclientconn causes serveGet to
signalFullyConsumedByteString to it, which is what it's waiting for

improve comment

update

http p2p proxy is now largely working

remove some debugging

wip

use a record to reduce the huge number of parameters

factored out proxyRequest

This will allow the HTTP server to keep a connection to a proxied
remote open, and only enter the Annex monad when handling a client
request.

http server support for proxies, incomplete

Refactored git-annex-shell code so this can use checkCanProxy'.

At this point all that remains is opening a proxy connection,
and using a proxy connection.

use annex+http for accessing proxies

Doesn't work yet on the http server side, which is throwing 502 bad
gateway.

less indent

p2phttp drop supports checking proof timestamps

At this point the p2phttp implementation is fully complete!

add common parameters to generic get API

Honestly this was just done to make the documentation correct. There's
no point in using these parameters. And they're optional.

fix build without servant

testremote passes on p2phttp remote

expire lock after 10 minutes initially

Once keeplocked is called, the lock will expire at the end
of that call. But if keeplocked never gets called, this avoids
the lock persisting forever.

Remote.Git lockContent works with annex+http urls

Remote.Git removeKey works with annex+http urls

Does not yet handle drop proof lock timestamp checking.

update

update

progress meter for p2phttp storeKey

Remote.Git storeKey works with annex+http urls

Does not yet update progress meter.

update

more generic clientGet

fix slowloris timeout in hashing resume of download of large file

Hash the data that is already present in the file before connecting to
the http server.

fix meter display on resume

It still needs to be offset, otherwise on resume from 80% it will
display 1%..20%.

Seems that this bug must have affected P2P.Annex as well where it runs
this code, but apparently it didn't affect it in a very user-visible
way. Maybe the transfer log file was updated incorrectly?

Remote.Git retrieveKeyFile works with annex+http urls

This includes a bugfix to serveGet, it hung at the end.

factor out resumeVerifyFromOffset

avoid needing ifdefs when using P2P.Http.Client

Merge branch 'master' into httpproto

Propagate --force to git-annex transferrer

And other child processes.

bug

cache credentials for p2phttp in memory

remove uuid from url when calling git credential

The user doesn't want to see a uuid in the prompt.

Also, when a http server is proxying, multiple remotes will have the
same base url, and the same password will work for them all.

use git credential when p2phttp needs auth

p2phttp support https

add --bind option and listen to both ipv4 and ipv6 by default

protocol version fallback on 404

and prettified errors

Remote.Git checkpresent works with annex+http urls.

factor out http server and client into own modules

To avoid a cycle when Remote.Git uses the client.

started wiring p2phttp into Remote.Git

but we have a cycle, ugh

require a valid uuid at the end of an annex+http url

add remoteAnnexP2PHttpUrl to RemoveGitConfig

This is always parsed, when building without servant, a Baseurl is not
generated, and users of it will need to fail.

starting support for remote.name.annexUrl set to annex+http

In this case, Remote.Git should not use that url for all access to
the repository. It will only be used for annex operations, which isn't
done yet.

update servant build flag

make it work when building w/o assistant

update

annex+http urls

refactor

fix build

update

change default port to 9417

Port 80 would need root, not a good idea, so pick something that might
work by default.

9418 is git protocol's port. 9419 is used by something, but nothing
known uses 9417, so it's as good a default as any.

exit cleanly on eg, failure to bind socket

P2P protocol is finalized

make keeplocked client UUID optional

It's not used, but it makes the protocol simpler if this parameter can
always be sent.

locking over http basically working

make serveKeepLocked check auth just to be safe

implement clientLockContent

implemented serveLockContent (untested)

improve clientKeepLocked

variable name

refactor

finished clientGet validity checking

improve clientGet types

fix TMVar left full crash affecting servePutOffset

Problem is that whatever is reading from the TMVar may not have read
from it yet before the client writes the next thing to it.

implemented servePutOffset and clientPutOffset

But, it's buggy: the server hangs without processing the VALIDITY,
and I can't seem to work out why. As far as I can see, storefile
is getting as far as running the validitycheck, which is supposed to
read that, but never does.

This is especially strange because what seems like the same protocol
doesn't hang when servePut runs it. This made me think that it needed
to use inAnnexWorker to be more like servePut, but that didn't help.

Another small problem with this is that it does create an empty
.git/annex/tmp/ file for the key. Since this will usually be used in
combination with servePut, that doesn't seem worth worrying about much.

fix crash in servePut when client sends from too high offset

Have to consume all the content from the client, but force an Invalid
result, so it will just get discarded.

clientPut seeking to offset

avoid padding in servePut

This means that when the client sends a truncated data to indicate
invalidity, DATA is not passed the full expected data. That leaves the
P2P connection in a state where it cannot be reused. While so far, they
are not reused, they will be later when proxies are supported. So, have
to close the P2P connection in this situation.

avoid padding in clientPut

Instead truncate when necessary to indicate invalid content was sent.
Very similar to how serveGet handles it.

fix releasing of p2p connection

avoid padding content in serveGet

Always truncate instead. The padding risked something not noticing the
content was bad and getting a file that was corrupted in a novel way
with the padding "X" at the end. A truncated file is better.

servePut and clientPut implementation

Made the data-length header required even for v0. This simplifies the
implementation, and doesn't preclude extra verification being done for
v0.

The connectionWaitVar is an ugly hack. In servePut, nothing waits
on the waitvar, and I could not find a good way to make anything wait on
it.

Added a comment: git-annex for managing music

Added a comment

Added a comment: `git annex sync --ff-only`

Added a comment: Also Serveo.net

Added a comment

Added a comment

Added a comment

reporting FTBFS on windows

Added a comment

Added a comment