Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Firewall access rules for a zone #15

Closed
geoffreak opened this issue Sep 27, 2016 · 11 comments
Closed

Firewall access rules for a zone #15

geoffreak opened this issue Sep 27, 2016 · 11 comments

Comments

@geoffreak
Copy link

Any chance of the firewall access rules for a zone API being added to this library?
https://api.cloudflare.com/#firewall-access-rule-for-a-zone-list-access-rules

@terinjokes terinjokes modified the milestone: Release Version 2 Dec 30, 2016
@terinjokes terinjokes removed this from the Release Version 2 milestone Jun 21, 2017
@Cherry
Copy link

Cherry commented Aug 3, 2018

Any progress on this being added? We'd love to see this included.

@terinjokes terinjokes self-assigned this Aug 3, 2018
@terinjokes
Copy link
Contributor

@Cherry This is a WIP change I might abandon, but can you give b6a097c a shot and see if it works for you? Among other changes I'm trying out, it adds accountAccessFirewall, userAccessFirewall, and zoneAccessFirewall to the constructed client.

@terinjokes
Copy link
Contributor

@Cherry Another ping to see if you can try out the pushed branch.

@Cherry
Copy link

Cherry commented Sep 13, 2018

Sorry for the delay Terin. I'll take a look at the changes this week and let you know how it goes.

@Cherry
Copy link

Cherry commented Sep 18, 2018

I finally had chance to test this, specifically the zoneAccessFirewall and accountAccessFirewall methods and they seem to work as expected. Thanks for the changes @terinjokes!

@terinjokes
Copy link
Contributor

Great. I'll see to finishing the codegen and getting it out in a release soon.

@nosovk
Copy link

nosovk commented Jul 23, 2020

it seems that currently accountAccessFirewall is removed from master branch. Any plans to revive it?

@terinjokes
Copy link
Contributor

I don't believe accountAccessFirewall was ever on master. I have no objections to merging it in if there's PRs.

@nosovk
Copy link

nosovk commented Jul 27, 2020

@terinjokes PR is done, could we merge?

@nosovk
Copy link

nosovk commented Aug 7, 2020

Another PR is done
#80

@nosovk
Copy link

nosovk commented Jan 18, 2023

You may use this PR by adding

"cloudflare": "github:cloudflare/node-cloudflare#pull/78/head"

to yours package.json

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

No branches or pull requests

5 participants