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

Modify the existing process of LocalSocket to support LocalSocket functionality #11778

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

wangchen61698
Copy link
Contributor

@wangchen61698 wangchen61698 commented Feb 26, 2024

Summary

Modify the existing process of LocalSocket to support LocalSocket functionality

Impact

localsocket

Testing

Refers to The Linux Programming Interface(https://www.man7.org/tlpi/index.html) chapter 57(sockets:UNIX DOMAIN) and their demo (us_xfr_sv.c & us_xfr_cl.c & ud_ucase_sv.c & ud_ucase_cl.c)
resouce code:
localsocket_tcp.zip
localsocket_udp.zip
test result:
TCP localsocket:
e2877ae2-a6cd-4fe0-9900-aaf546d04d76
UDP localsocket:
e6ba6f10-6117-4f7f-b9b4-9b993e4d5685

@wangchen61698 wangchen61698 changed the title Localsocket Modify the existing process of LocalSocket to support LocalSocket functionality Feb 26, 2024
@acassis
Copy link
Contributor

acassis commented Feb 26, 2024

@wangchen61698 there are many modifications in many files and you didn't include any comment about Testing. Did you test it? How did you test it? Please include more information here, include commit message and if possible instructions to let other people to test it too.

@wangchen61698
Copy link
Contributor Author

@wangchen61698 there are many modifications in many files and you didn't include any comment about Testing. Did you test it? How did you test it? Please include more information here, include commit message and if possible instructions to let other people to test it too.

I will complete the information later

@wangchen61698
Copy link
Contributor Author

wangchen61698 commented Feb 28, 2024

@wangchen61698 there are many modifications in many files and you didn't include any comment about Testing. Did you test it? How did you test it? Please include more information here, include commit message and if possible instructions to let other people to test it too.
@acassis

The testing information has been supplemented

@xiaoxiang781216 xiaoxiang781216 marked this pull request as draft February 28, 2024 18:19
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.

None yet

2 participants