1 Commits

Author SHA1 Message Date
Raúl Montejo
f00a96c40b Keep the Cloudflare token out of stack config
pulumi config set --secret writes ciphertext into Pulumi.dev.yaml, which is
committed, alongside the encryptionsalt. This stack's secrets provider is a
passphrase, and the passphrase is the empty string documented three lines above
it in the README. That combination is not encryption — it is the token spelled
differently, readable by anyone with access to this repo.

pulumi-cloudflare falls back to CLOUDFLARE_API_TOKEN when cloudflare:apiToken
is absent from config, so __main__.py is unchanged and so is pulumi up. The
token simply stops living in git.

The empty passphrase is left as it is. It protects nothing precisely because
nothing secret is in config any more, and that is what makes it harmless; the
notes now say to change the secrets provider before adding a secret at rest.

This does NOT un-expose the existing token. The value is still readable in this
branch's history at 94dd775, and history rewriting would not help either, since
it has been pushed since 2026-09-09. Only rotating the credential closes that;
notes/CLOUDFLARE.md now covers how, including checking Last used before
deleting the token, which is the only record of whether it was ever used.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-09-17 23:27:08 -04:00
4 changed files with 64 additions and 4 deletions

View File

@@ -1,5 +1,3 @@
encryptionsalt: v1:/EuCFtJK3aQ=:v1:gdT9CSE37lfsFVwI:3Q18OoBOcARW9XQiXzB+U091ve8PxA==
config:
cloudflare:apiToken:
secure: v1:SpC3Q/LEQboSQqka:B4QZfgnwvnr4F//aJkpHWj3TQgrZ8J0zbua9VKYAhLjsOerzo4fXzi6OS5U9GIVe2rgSD61rBODMDFFr5VVYMnMgfjby
fctc-website-under-construction:accountId: 73c0883bcde89365097896c8390fe751

View File

@@ -16,11 +16,29 @@ pulumi login --local # this keeps Pulumi's state in a local file. see ./notes/
export PULUMI_CONFIG_PASSPHRASE=""
pulumi stack init dev # setting the passphrase to an empty string for now. see ./notes/PASSPHRASE.md
pulumi config set cloudflare:apiToken --secret # see ./notes/CLOUDFLARE.md
export CLOUDFLARE_API_TOKEN=... # NOT pulumi config. see ./notes/CLOUDFLARE.md
pulumi config set accountId # see ./notes/CLOUDFLARE.md
```
### The Cloudflare token goes in the environment, not the config
`pulumi config set --secret` writes the value into `Pulumi.dev.yaml` as ciphertext, and
that file is committed. With the local backend the stack's secrets provider is a
passphrase, and this stack's passphrase is the empty string, documented three lines above.
Committed ciphertext plus a committed salt plus a documented passphrase is not encryption
— it is the token, spelled differently, readable by everyone with access to this repo.
`pulumi-cloudflare` reads **`CLOUDFLARE_API_TOKEN`** from the environment when
`cloudflare:apiToken` is absent, so nothing in `__main__.py` changes and neither does
`pulumi up`. The token just stops living in git.
**The empty passphrase is not the problem and does not need fixing.** It protects nothing
because nothing secret is in config any more, which is the arrangement that makes it
harmless. It becomes a problem again the moment someone runs `pulumi config set --secret`
— so if this stack ever does need a secret at rest, change the secrets provider first.
See `./notes/PASSPHRASE.md`.
### Brind it up
```
pulumi up

View File

@@ -28,3 +28,34 @@ https://dash.cloudflare.com/profile/api-tokens
8. Click "Create Token"
9. The token will only be displayed once.
### Where the token goes
**Into the environment, never into `pulumi config`.**
```bash
export CLOUDFLARE_API_TOKEN=...
pulumi up
```
`pulumi-cloudflare` falls back to `CLOUDFLARE_API_TOKEN` when `cloudflare:apiToken` is
not set in config, so this needs no change to `__main__.py`.
Do **not** run `pulumi config set cloudflare:apiToken --secret`. On this stack that writes
ciphertext into the committed `Pulumi.dev.yaml`, next to a committed salt, unlocked by a
passphrase documented in `../README.md`. See `./PASSPHRASE.md`.
For anything unattended later, put it in a GitHub Actions secret and export it in the job
rather than committing it in any form.
### If a token has been exposed
Rotating the value is the only fix — rewriting git history does not help, because anyone
with repo access has already been able to read it for as long as it was pushed.
1. https://dash.cloudflare.com/profile/api-tokens, signed in as the user who created it
(tokens are user-scoped, so only that user sees them).
2. Check **Last used** before doing anything — deleting the token destroys the only
record of whether it was used.
3. **Roll** to keep the token and issue a new value, or **Delete** and create a fresh one
from the steps above.

View File

@@ -1,3 +1,16 @@
## The passphrase
The `dev` stack uses Pulumi's passphrase secrets provider with an **empty passphrase**,
set in the bootstrap steps in `../README.md`.
That is fine only while **no secret is stored in stack config**. An empty passphrase plus
the `encryptionsalt` committed in `Pulumi.dev.yaml` means any `secure:` value in that file
is recoverable by anyone who can read this repo. The Cloudflare token therefore lives in
`CLOUDFLARE_API_TOKEN` in the environment instead — see `./CLOUDFLARE.md`.
**Before putting any secret in stack config, change the secrets provider.** Otherwise the
value is public to everyone with repo access the moment it is committed.
## Updating the passphrase
pulumi stack change-secrets-provider passphrase
pulumi stack change-secrets-provider passphrase