mirror of
https://github.com/asciinema/asciinema.git
synced 2026-04-25 16:05:52 +03:00
[PR #187] [MERGED] adding support to loading API token from an environment variable #405
Labels
No labels
bug
compatibility
feature request
fit for beginners
help wanted
hosting
idea
improvement
packaging
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/asciinema#405
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/asciinema/asciinema/pull/187
Author: @samdmarshall
Created: 12/28/2016
Status: ✅ Merged
Merged: 1/28/2017
Merged by: @ku1ik
Base:
master← Head:token-environment-variable📝 Commits (4)
0919339adding support to loading API token from an environment variable to remove secrets from config file85ce06bupdating based on requested code-review changes86289b2fixing typo!6e76e8ecorrecting code logic📊 Changes
1 file changed (+2 additions, -2 deletions)
View changed files
📝
asciinema/config.py(+2 -2)📄 Description
This addresses #185, by adding a new fallback method of an environment variable named
ASCIINEMA_TOKENto store the user's API token so that sensitive information doesn't have to be kept in plaintext on disk.🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.