64 lines
1.8 KiB
Markdown
64 lines
1.8 KiB
Markdown
# Testing Notes
|
|
|
|
## Monitor Script - Ctrl+C Fix
|
|
|
|
**Issue**: Script wouldn't stop with Ctrl+C
|
|
**Fix**: Simplified signal handling with `os.Exit(0)` in goroutine
|
|
**Status**: ✅ Fixed in latest build
|
|
|
|
Test with:
|
|
```bash
|
|
./bin/monitor-auth
|
|
# Press Ctrl+C - should exit immediately
|
|
```
|
|
|
|
## Authentication Behavior
|
|
|
|
**Initial Assumption**: Kosmi auto-creates accounts (INCORRECT)
|
|
**Reality**: Standard login - invalid credentials fail as expected
|
|
|
|
The initial test with `email@email.com` / `password` happened to work because:
|
|
- That account already existed
|
|
- Or it was a test account
|
|
- NOT because Kosmi auto-creates accounts
|
|
|
|
**Actual Behavior**:
|
|
- Valid credentials → login success
|
|
- Invalid credentials → login failure (needs verification via monitoring)
|
|
- Standard authentication flow
|
|
|
|
## Next Steps
|
|
|
|
1. **Run monitoring with real login**:
|
|
```bash
|
|
./bin/monitor-auth -login
|
|
# Log in with actual credentials
|
|
# Review auth-monitor.log for API format
|
|
```
|
|
|
|
2. **Verify auth API format**:
|
|
- Check POST requests to engine.kosmi.io
|
|
- Look for login mutation structure
|
|
- Verify token format and expiry
|
|
- Document actual API response
|
|
|
|
3. **Update auth.go if needed**:
|
|
- Adjust GraphQL mutations to match actual API
|
|
- Update token parsing logic
|
|
- Test with real credentials
|
|
|
|
4. **Test reconnection**:
|
|
- Run bridge with auth
|
|
- Simulate network failure
|
|
- Verify automatic reconnection
|
|
- Check token refresh works
|
|
|
|
## Files Updated
|
|
|
|
- ✅ `cmd/monitor-auth/main.go` - Fixed Ctrl+C handling
|
|
- ✅ `bridge/kosmi/auth.go` - Removed incorrect comments
|
|
- ✅ `matterbridge.toml` - Removed incorrect warning
|
|
- ✅ `cmd/monitor-auth/README.md` - Removed auto-registration section
|
|
- ✅ `QUICK_START_AUTH.md` - Removed auto-registration references
|
|
- ✅ Deleted `AUTH_NOTES.md` - Contained incorrect information
|