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

support gitlab token via env var #7023

Merged
merged 1 commit into from
Sep 16, 2021
Merged

support gitlab token via env var #7023

merged 1 commit into from
Sep 16, 2021

Conversation

calebcartwright
Copy link
Member

Was banging my head against the desk trying to figure out why the Shields.io prod environment wasn't sending the auth info with gitlab.com bound requests... only to discover I left out this crucial piece 🤦‍♂️

I've been meaning to suggest some documentation for adding new server secrets, as I think it'd be a helpful reference for folks, plus I've clearly forgotten it myself on more than one occasion!

@shields-ci
Copy link

Messages
📖 ✨ Thanks for your contribution to Shields, @calebcartwright!

Generated by 🚫 dangerJS against 6d964f2

@chris48s
Copy link
Member

Yeah docs on this are a good shout 👍 I think a new config var usually needs adding to

  • config/custom-environment-variables.yml
  • config/default.yml
  • core/server/server.js (schema)
  • relevant docs (usually doc/server-secrets.md )

so there's definitely some scope to mess that up

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.

4 participants