-
Notifications
You must be signed in to change notification settings - Fork 20
Open
Labels
Description
- for MX queries - dnscommon.r2py doesn't create dictionary from packet received from DNS. It gives an IndexError because of https://github.com/SeattleTestbed/seattlelib_v2/blob/master/dnscommon.r2py#L609 function _parse_address as it is limit to IPv4 address only. So, using wireshark and using same technique for the PTR, NS query it can be resolved.
User traceback:
"dylink.r2py", line 546, in <module>
"dylink.r2py", line 407, in dylink_dispatch
"dylink.r2py", line 520, in evaluate
"librepy.r2py", line 153, in <module>
"dylink.r2py", line 446, in _dylink_dispatch
"dylink.r2py", line 407, in dylink_dispatch
"dylink.r2py", line 520, in evaluate
"ut_dnsquery_recordsmx.r2py", line 11, in <module>
"ut_dnsquery_recordsmx.r2py", line 8, in test_recordsMX
"dnsquery.r2py", line 243, in recordsMX
"dnsquery.r2py", line 140, in dns_connection
"dnscommon.r2py", line 406, in convert_packet_to_dictionary
"dnscommon.r2py", line 459, in _read_answer_section
"dnscommon.r2py", line 500, in _read_single_answer
"dnscommon.r2py", line 703, in _parse_address
Exception (with type 'exceptions.IndexError'): string index out of range
---
- for 'AAAA' records - dnscommon doesn't support 'AAAA' i.e. IPv6 records yet. Hence a new function like
https://github.com/SeattleTestbed/seattlelib_v2/blob/master/dnscommon.r2py#L569 should be created for IPv6
Few MX queries has 'AAAA' records in Additional section for eg. dig gmail.com mx. So, for implementing complete MX records we need 'AAAA' support too.
Reactions are currently unavailable