Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introless multiintro yaml config.1 #260

Closed

Conversation

@david415
Copy link
Member

david415 commented Apr 19, 2016

No description provided.

kytvi2p and others added 30 commits Feb 18, 2012
author: Faruq (original darcs patches available from
https://tahoe-lafs.org/trac/tahoe-lafs/ticket/68)
Here we adapt the multi-introducer patch (ticket #68)
to accomodate the new sequence number and nonce scheme from ticket #1767.
We pass the current_seqnum and current_nonce into IntroducerClient's publish method
instead of that publish method calling the passed in _sequencer function to generate
that sequnum/nonce pair.

Manually resolved conflicts in these files:
	src/allmydata/client.py
        src/allmydata/introducer/client.py
        src/allmydata/introducer/interfaces.py
Leaving these files to Leif's commit to be merged later:
	src/allmydata/test/test_web.py
	src/allmydata/web/root.py
	src/allmydata/web/welcome.xhtml
note: this is a non-working state

Conflicts:
	src/allmydata/client.py
	src/allmydata/test/test_web.py
	src/allmydata/web/root.py
	src/allmydata/web/welcome.xhtml
Conflicts:
	src/allmydata/client.py
	src/allmydata/test/test_web.py
	src/allmydata/web/root.py
	src/allmydata/web/welcome.xhtml
Renaming this led to discovering a subtle bug in test.common - it was testing
the truthiness of a list of bools, which is of course True when the list is
[False].
I think that for v1 announcements the seqnum and nonce are ignored anyway.
this patch was sent to me by an anonymous contributor
david415 added 27 commits Mar 30, 2016
here we also:
 - Make IntroducerClient receive a subscribe_only bool arg
in the constructor
 - Client loads a connections.yaml with introducers and static storage servers
@warner

This comment has been minimized.

Copy link
Member

warner commented Sep 9, 2016

superceded (mostly) by #338, but I think we need to port the "introducerless" bits to that branch (it only has multi-introducer right now)

@david415 david415 closed this Sep 14, 2016
@coveralls

This comment has been minimized.

Copy link

coveralls commented May 17, 2018

Coverage Status

Coverage decreased (-16.5%) to 75.238% when pulling 7f802e8 on david415:introless-multiintro_yaml_config.1 into 04f1a3c on tahoe-lafs:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.