-
Notifications
You must be signed in to change notification settings - Fork 11
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
[EN-848] Service request chain tracking #6
Conversation
tjoelsson
commented
Jan 12, 2016
- uuid in header and access log (@wenzong)
- access log in NCSA combined format
@imlazyone @wanleung @agirivera please review |
👍 |
So no IP then? |
As I understand, getting the IP would require passing it from the client. I don't think it's worth doing that. At least not right now. |
ZeroMQ takes care of the address, I don't think pyzmq expose the API to get endpoint. |
@wanleung @agirivera please review this. it's blocking work on another ticket |
@@ -37,6 +38,9 @@ def logger(self): | |||
|
|||
class LocalContext(object): | |||
|
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.
Missing doc block.
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.
@wenzong anything to document here?
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.
sure, I can add some docstring
Only found trivial suggestion. Looks good to go 👍 |
Add docstring for LocalContext
[EN-848] Service request chain tracking