Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Address : Known issue #36

Open
2 of 4 tasks
alagoutte opened this issue Mar 2, 2019 · 12 comments
Open
2 of 4 tasks

Address : Known issue #36

alagoutte opened this issue Mar 2, 2019 · 12 comments

Comments

@alagoutte
Copy link
Contributor

alagoutte commented Mar 2, 2019

There is some known issue with address cmdlet

  • Only support ipmask(subnet) address for the moment
  • Don't check if the interface exist when add or edit
  • search/match is make by PowerShell (via Where-Object) and not direclty using API
  • Don't check if the address is use before modify/remove

This issues will be fixed on a next release...

@alagoutte
Copy link
Contributor Author

Search/march is fixed on #58 and #62 and it will be available on next release

@hlroudjehane
Copy link

Your module has been incredibly helpful. Any chance an geography address update is coming out soon? Any way I can update the API myself for this is not supported yet?

@alagoutte
Copy link
Contributor Author

Hi @hlroudjehane thanks for feedback !

I make a quick look of geography address and it will be easy to add ! (need to add a new parameter like -country with the country code...)
if you have time, you can also update the API and make a PR !

What the use case ?

@hlroudjehane
Copy link

hlroudjehane commented Jun 29, 2022 via email

@hlroudjehane
Copy link

hlroudjehane commented Jun 29, 2022 via email

@alagoutte
Copy link
Contributor Author

It looks like the change should be made in this file/location correct?

Yes !

@alagoutte
Copy link
Contributor Author

after a very quick look a small PR for add #191

some example :

Add-FGTFirewallAddress -Name FGT-Country-FR -country FR

name                 : FGT-Country-FR
q_origin_key         : FGT-Country-FR
uuid                 : 33cf29c8-f7e3-51ec-6987-6871166d6ba9
type                 : geography
sub-type             : sdn
clearpass-spt        : unknown
macaddr              : {}
country              : FR
cache-ttl            : 0
sdn                  : 
fsso-group           : {}
interface            : 
obj-type             : ip
tag-detection-level  : 
tag-type             : 
dirty                : dirty
comment              : 
associated-interface : 
color                : 0
filter               : 
sdn-addr-type        : private
node-ip-only         : disable
obj-id               : 
list                 : {}
tagging              : {}
allow-routing        : disable
fabric-object        : disable

Add-FGTFirewallAddress -Name FGT-Country-US -country US

name                 : FGT-Country-US
q_origin_key         : FGT-Country-US
uuid                 : 610bc0b8-f7e3-51ec-5add-9d7c450d2e0f
type                 : geography
sub-type             : sdn
clearpass-spt        : unknown
macaddr              : {}
country              : US
cache-ttl            : 0
sdn                  : 
fsso-group           : {}
interface            : 
obj-type             : ip
tag-detection-level  : 
tag-type             : 
dirty                : dirty
comment              : 
associated-interface : 
color                : 0
filter               : 
sdn-addr-type        : private
node-ip-only         : disable
obj-id               : 
list                 : {}
tagging              : {}
allow-routing        : disable
fabric-object        : disable


Get-FGTFirewallAddress -filter_attribute type -filter_value geography | ft

name           q_origin_key   uuid                                 type      sub-type clearpass-spt macaddr country cache-ttl sdn
----           ------------   ----                                 ----      -------- ------------- ------- ------- --------- ---
FGT-Country-FR    FGT-Country-FR    33cf29c8-f7e3-51ec-6987-6871166d6ba9 geography sdn      unknown       {}      FR              0 
FGT-Country-US FGT-Country-US 610bc0b8-f7e3-51ec-5add-9d7c450d2e0f geography sdn      unknown       {}      US              0 

@hlroudjehane
Copy link

hlroudjehane commented Jun 30, 2022 via email

@alagoutte
Copy link
Contributor Author

I updated the 12 lines of code highlighted in address.ps1 to no resolve unfortunately. Is there another file that needs updating?

do you reforce the import after modified the file ? Import-Module PowerFGT -force

@hlroudjehane
Copy link

hlroudjehane commented Jun 30, 2022 via email

@alagoutte
Copy link
Contributor Author

Yes in the screenshot I sent you I ran an import. It doesn’t seem to recognize the new parameters added as they don’t auto show while type.

there is no screenshot on github issue...

you can try to clone my repo and use the feature branch

@hlroudjehane
Copy link

hlroudjehane commented Oct 11, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants