<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div style="" class="markdown-here-wrapper"
data-md-url="Thunderbird">
<p style="margin: 1.2em 0px ! important;">On 31/10/14 15:15, Jen
Linkova wrote:</p>
<blockquote style="margin: 1.2em 0px;border-left: 4px solid
rgb(221, 221, 221); padding: 0px 1em; color: rgb(119, 119, 119);
quotes: none;">
<p style="margin: 1.2em 0px ! important;">I’ve just found that
using Sagan library for analyzing UDP traceroute<br>
measurement results may produce incorrect results:
‘target_responded’<br>
variable is not be set to ‘True’ if the source address of the<br>
traceroute reply is not the target address of the traceroute,
but any<br>
other address that belongs to the node.</p>
</blockquote>
<p style="margin: 1.2em 0px ! important;">Hi there, and sorry
about the late reply on this. I’m the primary author on Sagan
and I’ve been on vacation for the past couple weeks.</p>
<p style="margin: 1.2em 0px ! important;">This email was the topic
of some debate in the office and we’ve arrived at a few
suggestions that I’d like some community feedback on if anyone
has two cents to throw this way:</p>
<p style="margin: 1.2em 0px ! important;">The proposal is to
remove <code style="font-family:
Consolas,Inconsolata,Courier,monospace;margin: 0px 0.15em;
padding: 0px 0.3em; white-space: pre-wrap; border: 1px solid
rgb(234, 234, 234); background-color: rgb(248, 248, 248);
border-radius: 3px; display: inline;">target_responded</code>
and introduce two new properties:</p>
<ul style="margin: 1.2em 0px;padding-left: 2em;">
<li style="margin: 0.5em 0px;">A boolean named <code
style="font-family:
Consolas,Inconsolata,Courier,monospace;margin: 0px 0.15em;
padding: 0px 0.3em; white-space: pre-wrap; border: 1px solid
rgb(234, 234, 234); background-color: rgb(248, 248, 248);
border-radius: 3px; display: inline;">destination_ip_responded</code>
to reflect the current behaviour of <code style="font-family:
Consolas,Inconsolata,Courier,monospace;margin: 0px 0.15em;
padding: 0px 0.3em; white-space: pre-wrap; border: 1px solid
rgb(234, 234, 234); background-color: rgb(248, 248, 248);
border-radius: 3px; display: inline;">target_responded</code></li>
<li style="margin: 0.5em 0px;">A boolean named something like <code
style="font-family:
Consolas,Inconsolata,Courier,monospace;margin: 0px 0.15em;
padding: 0px 0.3em; white-space: pre-wrap; border: 1px solid
rgb(234, 234, 234); background-color: rgb(248, 248, 248);
border-radius: 3px; display: inline;">last_hop_responded</code>
or just <code style="font-family:
Consolas,Inconsolata,Courier,monospace;margin: 0px 0.15em;
padding: 0px 0.3em; white-space: pre-wrap; border: 1px solid
rgb(234, 234, 234); background-color: rgb(248, 248, 248);
border-radius: 3px; display: inline;">target_responded</code>
representing whether the last hop was a response at all.</li>
</ul>
<p style="margin: 1.2em 0px ! important;">What do you (and the
others on this list) think of this? Making the change to Sagan
is trivial, but I’d rather only do something like this once.</p>
<p style="margin: 1.2em 0px ! important;">I’d also happily audit
and accept <a
href="https://github.com/RIPE-NCC/ripe.atlas.sagan">pull
requests</a> on GitHub if anyone fancies writing a patch
themselves.</p>
<div
title="MDH:T24gMzEvMTAvMTQgMTU6MTUsIEplbiBMaW5rb3ZhIHdyb3RlOjxicj4mZ3Q7IEkndmUganVzdCBmb3VuZCB0aGF0IHVzaW5nIFNhZ2FuIGxpYnJhcnkgZm9yIGFuYWx5emluZyBVRFAgdHJhY2Vyb3V0
ZTxicj4mZ3Q7IG1lYXN1cmVtZW50IHJlc3VsdHMgbWF5IHByb2R1Y2UgaW5jb3JyZWN0IHJlc3Vs
dHM6ICd0YXJnZXRfcmVzcG9uZGVkJzxicj4mZ3Q7IHZhcmlhYmxlIGlzIG5vdCBiZSBzZXQgdG8g
J1RydWUnIGlmIHRoZSBzb3VyY2UgYWRkcmVzcyBvZiB0aGU8YnI+Jmd0OyB0cmFjZXJvdXRlIHJl
cGx5IGlzIG5vdCB0aGUgdGFyZ2V0IGFkZHJlc3Mgb2YgdGhlIHRyYWNlcm91dGUsIGJ1dCBhbnk8
YnI+Jmd0OyBvdGhlciBhZGRyZXNzIHRoYXQgYmVsb25ncyB0byB0aGUgbm9kZS48YnI+PGJyPkhp
CiB0aGVyZSwgYW5kIHNvcnJ5IGFib3V0IHRoZSBsYXRlIHJlcGx5IG9uIHRoaXMuwqAgSSdtIHRo
ZSBwcmltYXJ5IGF1dGhvciAKb24gU2FnYW4gYW5kIEkndmUgYmVlbiBvbiB2YWNhdGlvbiBmb3Ig
dGhlIHBhc3QgY291cGxlIHdlZWtzLjxicj48YnI+VGhpcwogZW1haWwgd2FzIHRoZSB0b3BpYyBv
ZiBzb21lIGRlYmF0ZSBpbiB0aGUgb2ZmaWNlIGFuZCB3ZSd2ZSBhcnJpdmVkIGF0IGEKIGZldyBz
dWdnZXN0aW9ucyB0aGF0IEknZCBsaWtlIHNvbWUgY29tbXVuaXR5IGZlZWRiYWNrIG9uIGlmIGFu
eW9uZSBoYXMgCnR3byBjZW50cyB0byB0aHJvdyB0aGlzIHdheTo8YnI+PGJyPlRoZSBwcm9wb3Nh
bCBpcyB0byByZW1vdmUgYHRhcmdldF9yZXNwb25kZWRgIGFuZCBpbnRyb2R1Y2UgdHdvIG5ldyBw
cm9wZXJ0aWVzOjxicj4KPGJyPgoqIEEgYm9vbGVhbiBuYW1lZCBgZGVzdGluYXRpb25faXBfcmVz
cG9uZGVkYCB0byByZWZsZWN0IHRoZSBjdXJyZW50IGJlaGF2aW91ciBvZiBgdGFyZ2V0X3Jlc3Bv
bmRlZGA8YnI+CiogQSBib29sZWFuIG5hbWVkIHNvbWV0aGluZyBsaWtlIGBsYXN0X2hvcF9yZXNw
b25kZWRgIG9yIGp1c3QgCmB0YXJnZXRfcmVzcG9uZGVkYCByZXByZXNlbnRpbmcgd2hldGhlciB0
aGUgbGFzdCBob3Agd2FzIGEgcmVzcG9uc2UgYXQgCmFsbC48YnI+PGJyPldoYXQgZG8geW91IChh
bmQgdGhlIG90aGVycyBvbiB0aGlzIGxpc3QpIHRoaW5rIG9mIHRoaXM/wqAgCk1ha2luZyB0aGUg
Y2hhbmdlIHRvIFNhZ2FuIGlzIHRyaXZpYWwsIGJ1dCBJJ2QgcmF0aGVyIG9ubHkgZG8gc29tZXRo
aW5nIApsaWtlIHRoaXMgb25jZS48YnI+PGJyPkknZCBhbHNvIGhhcHBpbHkgYXVkaXQgYW5kIGFj
Y2VwdCBbcHVsbCAKcmVxdWVzdHNdKGh0dHBzOi8vZ2l0aHViLmNvbS9SSVBFLU5DQy9yaXBlLmF0
bGFzLnNhZ2FuKSBvbiBHaXRIdWIgaWYgCmFueW9uZSBmYW5jaWVzIHdyaXRpbmcgYSBwYXRjaCB0
aGVtc2VsdmVzLjxicj48YnI+"
style="height:0;width:0;max-height:0;max-width:0;overflow:hidden;font-size:0em;padding:0;margin:0;"></div>
</div>
</body>
</html>