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

feat: add ability to specify custom principal #84

Merged
merged 1 commit into from
Aug 25, 2023
Merged

Conversation

travis
Copy link
Member

@travis travis commented Aug 24, 2023

This is needed by one of the teams integrating with w3up.

w3up-client will already fail if you pass a different principal than the store is configured to use, so we don't need to add any extra validation logic - end-users can generate a key with npx ucan-key ed --json and set the W3_PRINCIPAL environment variable for at least the first invocation of w3 in a new environment.

Setting W3_PRINCIPAL on subsequent invocations is optional but will ensure the principal used by the client is always the same.

This is needed by one of the teams integrating with w3up. w3up-client will already fail if you pass a different principal than the store is configured to use, so we don't need to add any extra validation logic - end-users can generate a key with `npx ucan-key ed --json` and set the `W3_PRINCIPAL` environment variable for at least the first invocation of `w3` in a new environment. Setting `W3_PRINCIPAL` on subsequent invocations is optional but will ensure the principal used by the client is always the same.
@travis travis merged commit 6115101 into main Aug 25, 2023
1 check passed
@travis travis deleted the feat/custom-principal branch August 25, 2023 17:15
travis pushed a commit that referenced this pull request Aug 25, 2023
🤖 I have created a release *beep* *boop*
---


##
[4.1.0](v4.0.0...v4.1.0)
(2023-08-25)


### Features

* add `space info` command to w3cli
([#83](#83))
([a72701c](a72701c))
* add ability to specify custom principal with the W3_PRINCIPAL
environment variable
([#84](#84))
([6115101](6115101))
* bump versions of `access` and `w3up-client`
([#81](#81))
([55de733](55de733))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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

Successfully merging this pull request may close these issues.

2 participants