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

Add categories in access policy #76

Closed
rnuyttens opened this issue Jun 13, 2020 · 9 comments
Closed

Add categories in access policy #76

rnuyttens opened this issue Jun 13, 2020 · 9 comments

Comments

@rnuyttens
Copy link

[**Is](url
categories.zip
)

I made a script for the creation of category in the access policy, however being beginning in python, could check if everything is good (and publish if you want)

Check with FMC 6.6 and 6.5

Best regards

@rnuyttens rnuyttens changed the title Add categories in access rules Add categories in access policy Jun 13, 2020
@daxm
Copy link
Collaborator

daxm commented Jun 15, 2020

Not saying this is wrong but it does break the methodology that we've been using for fmcapi. Each class translates directly to an API feature from the FMC. I don't see a specific API feature for "RuleCategories". I'd suggest you re-work it into the AccessRules class.

@caballero-oscuro
Copy link

Hi daxm,

Seeing as this is an open issue, am I to assume that there isn't a way to account for 'categories' already present in the 'AccessRules' class?

@daxm
Copy link
Collaborator

daxm commented Jul 27, 2020 via email

@caballero-oscuro
Copy link

No worries! Thanks for the response. Also, fmcapi has been a HUGE help to me and I just wanted to say thanks for doing this!

@daxm
Copy link
Collaborator

daxm commented Jul 27, 2020 via email

@kell4now
Copy link
Collaborator

I added support for categories quite some time back for ACP rules:
#31

@caballero-oscuro
Copy link

I added support for categories quite some time back for ACP rules:
#31

You're right, I do see that support added. However, I was only able to get it to work by adding "category" to:

VALID_FOR_KWARGS = VALID_JSON_DATA + [
"acp_id",
"acp_name",
"insertBefore",
"insertAfter",
"section",
"category"
]
It seems to be missing from the list. I'm still learning how this all works, so if I'm mistaken let me know. Thanks!

@kell4now
Copy link
Collaborator

@caballero-oscuro - ok, not sure why it was missing from that list. You can go ahead and create a branch add that in and submit a PR :)

@daxm
Copy link
Collaborator

daxm commented Jul 28, 2020

I just added it to Master and pushed to GitHub.

@daxm daxm closed this as completed Jul 28, 2020
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

4 participants