Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jun 9, 2025. It is now read-only.

feat: fetch secrets from OneGraph as-needed #107

Conversation

dwwoelfel
Copy link
Contributor

@dwwoelfel dwwoelfel commented Aug 2, 2021

Fetches secrets on each usage of withSecrets using the ONEGRAPH_AUTHLIFY_TOKEN. This works around the problem of needing to redeploy to update the secrets

  • I have read the contribution guidelines.
  • The status checks are successful (continuous integration). Those can be seen below.

@dwwoelfel dwwoelfel changed the title Fetch secrets from OneGraph as-needed feat: fetch secrets from OneGraph as-needed Aug 2, 2021
Copy link
Contributor

@ascorbic ascorbic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good! Just a few comments.


const formatSecrets = (result) => {
const services =
result.data && result.data.me && result.data.me.serviceMetadata && result.data.me.serviceMetadata.loggedInServices
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think optional chaining is what I miss most when not transpiling 😁

@ascorbic ascorbic merged commit 0682771 into netlify:feat/secrets-handling Aug 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants