ndejong / pfsense_fauxapi

REST based API interface for pfSense 2.3.x and 2.4.x to facilitate devops
Apache License 2.0
354 stars 61 forks source link

New API for network rules #59

Open slayercat opened 4 years ago

slayercat commented 4 years ago

Closes #50

Signed-off-by: lilinzhe slayercat.subscription@gmail.com

Korvuss commented 4 years ago

would it be possible to merge this as its exactly what I am currently looking for !

slayercat commented 4 years ago

Hi, @Korvuss

You may want to review issues #56 .

Thanks.

ndejong commented 4 years ago

Hi @slayercat pardon the long delay in my getting to this.

The code is well written, but you are re-declaring the namespace in an unexpected location

This is messy and almost looks as if it is an accidental piece of code that has crept in at development time

Also - can you propose some tests for this? I'll add them to the Python client interface examples that get used in the test harness