-
Notifications
You must be signed in to change notification settings - Fork 58
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
Include parameter to indicate endpoint #117
Include parameter to indicate endpoint #117
Conversation
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #117 +/- ##
===========================================
+ Coverage 0 58.77% +58.77%
===========================================
Files 0 4 +4
Lines 0 570 +570
Branches 0 147 +147
===========================================
+ Hits 0 335 +335
- Misses 0 134 +134
- Partials 0 101 +101 ☔ View full report in Codecov by Sentry. |
54ce3b0
to
89ad6b4
Compare
9c3b517
to
b49e1e1
Compare
0904089
to
9dba1de
Compare
7e9b597
to
38dc999
Compare
0a794e7
to
58691c1
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.
This looks good overall, but please fix the indent to match the file you are changing: e.g.: it seems you are using 2 spaces in some places like socket.h
and tangd.c
(perhaps others as well)
58691c1
to
f5dd28a
Compare
26e8b86
to
086108d
Compare
Proposed changes have been implemented
3335e5f
to
52ef7a7
Compare
1e2eb9c
to
2bebffa
Compare
This change allows specifying -e option, so that endpoint parameter is used. This parameter gives the possibility to change URL where Tang listens, so that endpoint provided is prepended between host port information and advertisement/recovery suffix (rec/adv). Without endpoint, advertisement URL is: http://localhost:port/adv Meanwhile, if using endpoint (-e this/is/endpoint), advertisement URL is: http://localhost:port/this/is/endpoint/adv For more information, check man page Resolves: latchset#116 Signed-off-by: Sergio Arroutbi <[email protected]>
2bebffa
to
f65decf
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 for this work!
Resolves: #116