Github user yatsukhnenko commented on the issue:
https://github.com/apache/trafficserver/pull/804
[This](https://issues.apache.org/jira/browse/TS-2819) one?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/804
Yeah, file a Jira probably. I could have sworn we've had issues before
where loop detection kicked in when it shouldn't (e.g. you legitimately proxy
to yourself). But I can find an issue on it,
Github user yatsukhnenko commented on the issue:
https://github.com/apache/trafficserver/pull/804
> What do you think of a future feature (config) to disable this Via
checking completely? If I know there's not chance of a loop, why bother
checking the Via header?
@zwoop,
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/804
Did a quick test of this, and it looks good:
Via: http/1.1 fedora.ogre.com[f6f5162d-5a02-4d10-ac6e-35aede508fe5]
(ApacheTrafficServer/7.0.0)
---
If your project is set up for it,
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/804
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/337/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/804
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/443/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/804
@yatsukhnenko What do you think of a future feature (config) to disable
this Via checking completely? If I know there's not chance of a loop, why
bother checking the Via header? I guess one
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/804
I checked the core on this FreeBSD build, and it seems completely unrelated
to pretty much anything :).
```
(gdb) bt
#0 0x in ?? ()
#1 0x006ec34a
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/804
I suspect this failed for other reasons, trying again [approve ci].
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/804
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/335/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/804
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/441/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/804
[approve ci]
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user yatsukhnenko commented on the issue:
https://github.com/apache/trafficserver/pull/804
Thanks for explaining about API usage
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/804
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/333/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/804
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/439/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/804
It's an inefficiency for starters (you go through a C-> C++ wrapper layer,
for no good reason (remember, the public APIs are all C, whereas in the core,
you can use all the C++ objects and
Github user yatsukhnenko commented on the issue:
https://github.com/apache/trafficserver/pull/804
Don't quite understand why public API can't be used in internal code, but
ok, changed code to `Machine::instance()->uuid.getString()`
---
If your project is set up for it, you can reply
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/804
Almost, but not quite. :) All APIs that are prefixed TS are the public
APIs, and we don't use those in the internal code. Look at the log tags in
logging that uses the internal APIs for the
18 matches
Mail list logo