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

Introduce the XRF peer type #85

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

chazapis
Copy link

@chazapis chazapis commented Mar 2, 2018

XRF peers can now be defined in the xlxd.interlink file,
allowing linking of a local module to a remote using
the DExtra protocol. XRF peers are shown in the peer list,
along with their XLX siblings.

Changes affect only how the reflector initiates the link,
as xlxd already supports incoming DExtra links.
Each peer is associated internally with a remote client,
so code changes include distinguishing peer connect acks
from non-peer ones and handling timeouts differently for
peer clients to clean up the peer data structure.
The DExtra disconnect packet is now properly implemented.

XRF peers can now be defined in the xlxd.interlink file,
allowing linking of a local module to a remote using
the DExtra protocol. XRF peers are shown in the peer list,
along with their XLX siblings.

Changes affect only how the reflector initiates the link,
as xlxd already supports incoming DExtra links.
Each peer is associated internally with a remote client,
so code changes include distinguishing peer connect acks
from non-peer ones and handling timeouts differently for
peer clients to clean up the peer data structure.
The DExtra disconnect packet is now properly implemented.
@ravny
Copy link

ravny commented Mar 11, 2018

Hi,

so if I want to try XRF linking from XLX reflector I just change files and restart service or I need to do something more?
Thank you

@chazapis
Copy link
Author

You just need to add the respective configuration in xlxd.interlkink. There are instructions in the file.

@sixfootseven
Copy link

sixfootseven commented Jun 4, 2018

Up and running now on my test server... can we do multiple module links between two reflectors?

@chazapis
Copy link
Author

chazapis commented Jun 5, 2018 via email

@w1bsb
Copy link

w1bsb commented Jun 10, 2018

I wonder if it would be possible for this feature to be modified so multiple links between two reflectors could be achieved?

@sixfootseven
Copy link

sixfootseven commented Jun 10, 2018 via email

@insanity213
Copy link

Running your code on my reflector, only have 2 XRF links but it's working perfectly so far. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants