Skip to content

feat: add SynologySignalHandler to Synology transfer extension#1484

Merged
mengyushen merged 1 commit into
dtinit:masterfrom
SynologyOpenSource:synology-add-signal-handler
Mar 13, 2026
Merged

feat: add SynologySignalHandler to Synology transfer extension#1484
mengyushen merged 1 commit into
dtinit:masterfrom
SynologyOpenSource:synology-add-signal-handler

Conversation

@simonxander
Copy link
Copy Markdown
Collaborator

Implemented SynologySignalHandler to manage job lifecycle signals (e.g., job started or completion) within the Synology transfer extension. This ensures that the Synology C2 API is notified of the final state of a transfer job, allowing for better synchronization.

Changes

  • New Signal Handler: Added SynologySignalHandler to process and retry job lifecycle signals using the
    RetryingCallable framework.
  • Service Integration: Added sendJobSignal to SynologyDTPService to handle the actual HTTP POST request to the
    Synology C2 API.
  • Extension Update: Modified SynologyTransferExtension to register the new SignalHandler during initialization.
  • Configuration & Models:
    • Updated synology.yaml to include the new /import/job/signal endpoint.
    • Updated C2Api and its ApiPath inner class to support the new signal path.
  • Testing:
    • Added SynologySignalHandlerTest to verify successful signal transmission and the retry mechanism on failure.
    • Updated TestConfigs to include the signal path for existing tests.

Implemented SynologySignalHandler to manage job lifecycle signals (e.g., job completion or failure).
- Added SynologySignalHandler to handle and retry job lifecycle signals.
- Added sendJobSignal to SynologyDTPService to communicate with the Synology C2 API.
- Updated SynologyTransferExtension to register the new signal handler.
- Configured the new /import/job/signal endpoint in synology.yaml.
- Added SynologySignalHandlerTest to verify signal sending and retry logic.
@mengyushen mengyushen removed their assignment Mar 12, 2026
@mengyushen mengyushen self-requested a review March 13, 2026 02:36
@mengyushen mengyushen merged commit f17ce32 into dtinit:master Mar 13, 2026
6 checks 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.

2 participants