regorus icon indicating copy to clipboard operation
regorus copied to clipboard

[Lib] Implement net builtins

Open anakrish opened this issue 1 year ago • 8 comments

Ensure that cargo test -r --test opa -- netcidrcontains netcidrcontainsmatches netcidrexpand netcidrintersects netcidrisvalid netcidrmerge netcidroverlap netlookupipaddr passes.

anakrish avatar Jan 05 '24 19:01 anakrish

Hey @anakrish - I'm a regular contributor to the OPA project and some of the others pointed me over here. I'm really excited to discover this library and I'd love to help you get some of these builtins implemented... I've been looking for a chance to get some OSS commits in rust as I seem to be spending most of my time in Go projects.

I'm going to take a crack at net.cidr_contains to start. I'm happy to work through all of these networking-related ones in time if you'd like to assign the issue to me. Since I'm newish to Rust, I'll probably open PRs for one builtin at a time so I can get good feedback about style and approach.

I'm on OPA slack with the same username. Would love to chat more about your project and how I can help

tjons avatar May 11 '25 23:05 tjons

Hey @tjons, Welcome on board! It's really awesome to see a regular OPA contributor also take on features in Regorus!

If you are interested in Rust, #212 is also a great task to take on.

anakrish avatar May 14 '25 15:05 anakrish

Hey @tjons, By any chance are you available on Teams?

anakrish avatar May 19 '25 17:05 anakrish

@anakrish I'm not, sorry! Slack and my email is in my gh profile, happy to chat there. Still working on this as I've been busy, but having fun :)

tjons avatar May 24 '25 18:05 tjons

Hey @anakrish and @tjons,

Good to see that you started working on the missing net functions!

I'd also be interested in this feature, and also willing to help with the implementation. Since you were looking for an off-Github comms channel, it's not quite clear to me what's going on. Would you mind posting an update on the current state?

Cheers, Markus

burgerdev avatar Aug 06 '25 15:08 burgerdev

Hey @burgerdev - thanks for following along! It's been a little slow moving, as I've been learning the ropes of the project. net.cidr_is_valid merged this morning, and I'm working on net.cidr_contains today. I expect I'll have something up by the weekend...

tjons avatar Aug 06 '25 18:08 tjons

Hey @burgerdev, @tjons is taking the lead for net builtins. Please feel free to coordinate and take on other builtins; currently all the communication is happening via github issues.

anakrish avatar Aug 06 '25 23:08 anakrish

Thanks all, that's great! I'm mostly interested in net.cidr_contains - looking forward to it...

burgerdev avatar Aug 07 '25 07:08 burgerdev