Skip to content

Conversation

@netomi
Copy link
Member

@netomi netomi commented Jul 24, 2025

Currently, whenever a QuartApp is constructed, an apptoken.txt is generated in the app directory (which is the cwd most of the time). This is also true for tests, which creates additional files for nothing and create clutter in your working directory.

This PR adds a token_file parameter, to allow more fine-grained control:

  • if the parameter is not provided, it defaults to the current behavior of using an apptoken.txt in the app dir
  • if None is provided, no secret is read or stored from file
  • You can provide an alternate name or absolute path to use a different token_file

@netomi
Copy link
Member Author

netomi commented Jul 24, 2025

Additionally, I cleaned up existing test cases to not generate a token_file to avoid creating apptoken.txt files when running the tests.

@netomi netomi changed the title Make token_file configurable and optional Make token_file configurable Jul 24, 2025
@netomi netomi requested a review from gstein July 25, 2025 10:31
@gstein gstein merged commit e983d3f into main Aug 1, 2025
4 checks passed
@gstein gstein deleted the make-apptoken-configurable branch August 1, 2025 10:14
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