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

Add parsing tokens from files or environment variables #374

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ShiromMakkad
Copy link

@ShiromMakkad ShiromMakkad commented Jun 20, 2024

Issues

Solves #264 and #218

Info

This adds support for token parsing from environment variables with the format RATHOLE_{service name in uppercase}_TOKEN or RATHOLE_DEFAULT_TOKEN. It also adds token_file and default_token_file to client and server configs. The updated Readme has a more detailed description of the changes.

Testing

Tested using cargo test. I've also added a file inside valid_configs to test for the file token and a new test case for environment variable parsing.

Todo

  • Update the Docker Hub Readme with environment variable parsing.
  • Update the Chinese version of the Readme.

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.

1 participant