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 support for Access Service Tokens #521

Conversation

jacobbednarz
Copy link
Member

@jacobbednarz jacobbednarz commented Oct 17, 2019

Introduces the ability to manage Cloudflare Access Service Tokens via
Terraform.

See https://developers.cloudflare.com/access/service-auth/service-token/

Closes #209

Screenshots

Inactive sidebar

Active sidebar

Documentation

@ghost ghost added size/L kind/documentation Categorizes issue or PR as related to documentation. labels Oct 17, 2019
@jacobbednarz
Copy link
Member Author

Just waiting on the permissions to be updated for the CI user to use Access to get the tests passing.

The Access Service Tokens endpoints unfortunately don't support API
tokens (the default authentication mechanism used in the tests) so we
need to unset it during test runs until this is resolved. It throws a
very confusing "insufficient permissions" exception.

```
=== RUN   TestAccAccessServiceTokenCreate
--- FAIL: TestAccAccessServiceTokenCreate (0.56s)
    testing.go:569: Step 0 error: errors during apply:

        Error: error creating access service token: error from makeRequest: HTTP status 403: insufficient permissions

          on /opt/teamcity-agent/temp/buildTmp/tf-test595601124/main.tf line 2:
          (source code not available)

FAIL
```
@ghost ghost added size/XL and removed size/L labels Oct 22, 2019
@jacobbednarz jacobbednarz merged commit 9acb97b into cloudflare:master Oct 22, 2019
@jacobbednarz jacobbednarz deleted the add-support-for-access-service-tokens branch October 22, 2019 20:22
boekkooi-lengoo pushed a commit to boekkooi-lengoo/terraform-provider-cloudflare that referenced this pull request Feb 28, 2022
The documentation for this endpoint mentions "latency" in place of "rtt"
however the actual responses use the latter.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/documentation Categorizes issue or PR as related to documentation. size/XL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing service token support for Cloudflare access
2 participants