search

pierky / arouteserver

A tool to automatically build (and test) feature-rich configurations for BGP route servers.
https://arouteserver.readthedocs.org/
GNU General Public License v3.0
286 stars 46 forks source link

bgp.last doesn't support aggregated origins #34

Closed s1sfa closed 6 years ago

s1sfa commented 6 years ago

When a network that is advertising is aggregating like this using aggregation. bgp_path.last returns a 0 so the origin as checks naturally fail when checking AS0 agains the IRRDB lists.

In my generated bird.conf I replaced bgp_path.last with bgp_path.last_nonaggregated which didn't seem to cause any problems and fixed the rejected routes in my small set of routes. So naturally if I change every occurrence of bgp_path.last to bgp_path.last_nonaggregated in the whole repo it should fix this problem :)

pierky commented 6 years ago

Hi @s1sfa, thanks for filing this issue and for the PR. I've tested it and merged it, v0.19.1 is building right now and will be out in a few minutes.