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.

Sign up for GitHub

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

Jump to bottom

Fix/gateway pp fix #96

Merged
merged 2 commits into from
Feb 7, 2022
Merged

Fix/gateway pp fix #96

merged 2 commits into from
Feb 7, 2022

Conversation

mdumais-stratos
Copy link
Contributor

  • Send messages to SP directly instead of through the gateway PP when SP is available
  • Use TransferSendMessageToPPServ instead of TransferSendMessageToClient
  • Local ppList now has a map indexed by p2pAddress and one by networkAddress, so it's easier to fetch the information of a known PP node

qinystratos and others added 2 commits February 7, 2022 12:09
@qinystratos
- try sending msg to sp directly first before sending msg via a gatew…
76cee4f 
…ay pp

- replace all appearances of TransferSendMessageToClient with TransferSendMessageToPPServ
@mdumais-stratos
fix/ use local PP list to transfer messages if only network address i…
99bd0b2 
…s available
@alexstratos alexstratos merged commit dc67080 into dev Feb 7, 2022
@alexstratos alexstratos deleted the fix/gateway_pp_fix branch February 7, 2022 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexstratos alexstratos alexstratos approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mdumais-stratos @alexstratos @qinystratos