Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Oct 23 18:09
    thomas-mangin commented #927
  • Oct 23 18:08

    thomas-mangin on master

    change the type/subtype #927 (compare)

  • Oct 23 17:59
    thomas-mangin commented #929
  • Oct 23 17:59
    thomas-mangin labeled #929
  • Oct 23 17:59
    thomas-mangin labeled #929
  • Oct 23 17:58
    thomas-mangin assigned #929
  • Oct 23 17:58
    thomas-mangin commented #929
  • Oct 23 16:46
    juise commented #931
  • Oct 23 16:46
    thomas-mangin commented #930
  • Oct 23 16:46
    juise commented #931
  • Oct 23 16:45
    juise synchronize #931
  • Oct 23 15:49
    juise commented #930
  • Oct 23 15:48
    juise commented #930
  • Oct 23 15:40
    thomas-mangin commented #931
  • Oct 23 13:59
    thomas-mangin commented #931
  • Oct 23 13:42
    juise commented #930
  • Oct 23 13:42
    juise opened #931
  • Oct 23 13:39
    juise commented #930
  • Oct 23 13:29
    thomas-mangin commented #930
  • Oct 23 13:28
    thomas-mangin labeled #930
Thomas Mangin
@thomas-mangin
I am not using capabilities
so I am stuck but yes it would be the long and correct way - thank you for reminding me
finishing the patch ..
should be done soon
As long the the nice pear liqueur is not too harmful to my coding :wink2:
sanjmonkey
@sanjmonkey
:+1:
Thomas Mangin
@thomas-mangin
Ok - the patch is ready but it comes with a price attached :grin:
documentation for the MD5 and new per peer “listen” which allows to set per peer MD5
sanjmonkey
@sanjmonkey
a bottle of pear liqueur ?
Thomas Mangin
@thomas-mangin
LOL - no - that’s ok still plenty left
@sanjmonkey can I please have your full name for the changelog ?
sanjmonkey
@sanjmonkey
sure, Sandy Breeze
Thomas Mangin
@thomas-mangin
Thank - I only remembered your first name
sanjmonkey
@sanjmonkey
np!
Thomas Mangin
@thomas-mangin
pushed to my tree - as soon as CI reports all fine - pushing to main repo
done
sanjmonkey
@sanjmonkey
cloning!
Thomas Mangin
@thomas-mangin
You should be having a relaxing evening .. but who am I to judge :tongue:
sanjmonkey
@sanjmonkey
perhaps I cloned too early, no reply to md5 session :(
22:06:11.174545 IP <x.x.x.x>.40356 > <y.y.y.y>.179: Flags [S], seq 3786399209, win 29200, options [nop,nop,md5valid,mss 1460,nop,nop,sackOK,nop,wscale 7], length 0
Thomas Mangin
@thomas-mangin
hum ...
on the receiving exabgp you need to add
“listen <port>"
sorry
sanjmonkey
@sanjmonkey
oooh
Thomas Mangin
@thomas-mangin
“listen <port>;"
like you would do for passive
or hold-time
MD5 is a per peer setting
previously the only way to listen was via the global option exabgp.tcp.bind
it is not the case anymore
hence why the patch took a few hours
sanjmonkey
@sanjmonkey

i see new reactor message: Listening for BGP session(s) on <y.y.y.y>:179 with MD5

but no ack to this syn:
22:12:52.454633 IP <x.x.x.x>.33553 > <y.y.y.y>.179: Flags [S], seq 2573773506, win 29200, options [nop,nop,md5valid,mss 1460,nop,nop,sackOK,nop,wscale 7], length 0

passive side is listening as it should
Thomas Mangin
@thomas-mangin
passive : do not establish outgoing connection
so without using the global listening option a passive neighbour is as good as unconfigured
listen <port> accept incoming connection
so both options are orthogonals
you can have passive, passive + listen, listen
hum .. not sure why it does not work for you ...
I need to add an “active” or “port” option to set out on what port the peer will connect out ..
when I tried I only tested 1790 ( to not have to use root )
but it should make no difference
(and I am lying I did test 179 .. ) - tired
sanjmonkey
@sanjmonkey
ok, and 179 works for you?
Thomas Mangin
@thomas-mangin
yes
sanjmonkey
@sanjmonkey
me too (tired). let me look at it with a fresh pair of eyes tomorrow, perhaps I’ve missed something obvious!
Thomas Mangin
@thomas-mangin
ok - I should be online in the afternoon
sanjmonkey
@sanjmonkey
dont want to waste your time
Thomas Mangin
@thomas-mangin
feel free to grab me here