Skip to content

fix(storage): stabilize MySQL provider and resolve ATS network issues#66

Merged
MasamiYui merged 3 commits into
mainfrom
fix/storage-ats-stability
Feb 28, 2026
Merged

fix(storage): stabilize MySQL provider and resolve ATS network issues#66
MasamiYui merged 3 commits into
mainfrom
fix/storage-ats-stability

Conversation

@MasamiYui

Copy link
Copy Markdown
Member

Description

This PR addresses critical stability issues related to database initialization and network connectivity for the email gateway.

Key Changes

  • Database Stability:
    • Narrowed the configuration observation scope in StorageManager to prevent unnecessary re-initialization.
    • Added a 1-second debounce and dropFirst() to the storage configuration stream to avoid race conditions during app startup.
    • Replaced synchronous connection pool shutdown with an asynchronous approach in MySQLStorage to prevent main thread blocking.
  • Network Connectivity:
    • Added an App Transport Security (ATS) exception in Info.plist for the FastMail gateway IP (47.99.241.152) to allow HTTP connections for testing and deployment.
  • Bug Fixes:
    • Fixed a typo in SettingsView where the token save method was incorrectly referenced.

Verification

  • Verified that the app no longer crashes/freezes with NIO assertion errors during startup.
  • Verified that the email test button works correctly without being blocked by ATS.
  • Verified that changing MySQL settings no longer triggers multiple simultaneous re-connections.

- Extract markdown generation to MeetingTask
- Fix SettingsCategory enum conflict in SettingsView
- Improve email notification trigger logic in pipeline
- Clean up unused comments and improve logging
- avoid provider rebuild on app launch
- debounce mysql config changes and observe password state
- shut down mysql event loop asynchronously
- fix transcript tab call and email token save
- allow http gateway for configured host
@MasamiYui MasamiYui merged commit 4037b00 into main Feb 28, 2026
1 check passed
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