Skip to content

Conversation

ingalls
Copy link
Member

@ingalls ingalls commented Jul 11, 2025

Context

Merged in main - See: #712 for changes

chriselsen and others added 8 commits June 30, 2025 22:47
- Test admin creation with CLOUDTAK_ADMIN_USERNAME/PASSWORD
- Test skipping admin creation when credentials missing
- Removing the unused Config import
- Removing the unused err parameter in the catch block for the schema validation test
Works with CDK deployed CloudTAK layer.
Updated tests for working config.ts version
Updated documentation
@ingalls ingalls self-assigned this Jul 11, 2025
@ingalls
Copy link
Member Author

ingalls commented Jul 12, 2025

  • Need to write comprehensive tests before this gets merged
  • Need to reset process.env between test runs as the V8 context is shared

Fix debug output to use correct variable name in certificate length logging
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants