-
-
Notifications
You must be signed in to change notification settings - Fork 556
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
feat(rust): rename the port field into address and support change the address and the port #5894
feat(rust): rename the port field into address and support change the address and the port #5894
Conversation
f400554
to
cd1002b
Compare
cd1002b
to
7adb614
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @0x61nas , thank you so much for creating your first PR at Ockam! It looks pretty nice, just a small change regarding how to parse the service address 👍
Also, make sure to setup your github account so that your commits are signed. You can follow this guide or ask us if you have any question.
implementations/rust/ockam/ockam_app/src/shared_service/tcp_outlet/create.rs
Outdated
Show resolved
Hide resolved
implementations/typescript/ockam/ockam_app/src/routes/service/ServiceCreate.svelte
Outdated
Show resolved
Hide resolved
implementations/typescript/ockam/ockam_app/src/routes/service/ServiceCreate.svelte
Outdated
Show resolved
Hide resolved
543c99c
to
6bc9cb8
Compare
6bc9cb8
to
5c5726a
Compare
9c4a5c1
to
bb576d5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks perfect! Just a couple of minor changes and will be ready to get merged 💪
implementations/rust/ockam/ockam_app/src/shared_service/tcp_outlet/create.rs
Outdated
Show resolved
Hide resolved
bb576d5
to
ac37a0a
Compare
d4b97e4
to
307f65a
Compare
… address and the port The changes include: * Replace the Port field name to Address * Support the formats Host:Port, IP:Port and Port (defaulting to 127.0.0.1 aka `localhost`) * Adapt the app back-end to parse the address correctly. - Closes: build-trust#5685
307f65a
to
9b931e6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🚀 Thanks @0x61nas for an awesome first contribution 🥳
Sorry for the late PR, I had to take some time to parse the project structure tree.
Current behavior
The Share a service window only allows the user to define the service port and the host address it always defaults to
127.0.0.1
Proposed changes
Host:Port
,IP:Port
and Port (defaulting to 127.0.0.1)Checks