Linkerd2: Update the skip inbound and outbound ports config to support range
Currently, the skip-[inbound|outbound]-ports configuration is a comma-delimited string. It will be useful to update this to support a range of ports, for cases where the (inbound or outbound) ports are random and ephemeral.
ihcsim
All 5 comments
I'd like to take this one, if that's alright!
javaducky
on 23 Nov 2019
@javaducky Thanks for your interest! @dddanmar already mentioned that he'll like to take this on. Also, it might be a bit more involved that I first anticipated. Let me chat with the team about this first, and add more info here.
ihcsim
on 23 Nov 2019
Please disregard the above commit. I should not have included the issue number in the commit message as it was automatically linked. I'm simply coding a solution for "practice."
javaducky
on 25 Nov 2019
@ihcsim @dddanmar Just in case, I have a fully functional patch with unit tests if you'd like it. Hopefully I don't sound pushy as that's not my intent...I'm just excited about contributing. :)
javaducky
on 27 Nov 2019
No issues from my side!
dddanmar
on 27 Nov 2019
Related issues
adleong
路
4Comments
briansmith
路
4Comments
steve-fraser
路
4Comments
skalinets
路
3Comments
vikas027
路
4Comments
Most helpful comment
@javaducky Thanks for your interest! @dddanmar already mentioned that he'll like to take this on. Also, it might be a bit more involved that I first anticipated. Let me chat with the team about this first, and add more info here.