Skip to content

python: backward compatibility issue #180

@sim590

Description

@sim590

The whole benchmark is broken by 768bfa5 since the SockAddr is not used by the benchmark and no "legacy" signature (with address as string) is still available in the python API for bootstraping 😒.

This is the case for benchmark, but also all python code which have already been written before 768bfa5. For instance, the pingpong script is broken. I think that some of ring's dht small swarm is using python bindings also. This is dangerous.

This kind of changes should be made more carefuly in the future since we never know who may use the python bindings also and we don't want to just drop support for API as fundamental as DhtRunner.bootstrap. I suggest, a legacy signature should be provided. If we want to deprecate it, then we could add a message on standard error specifying the matter, but anyhow I don't think it's good idea to remove this signature before a long time (couple of weeks or months).

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions