Skip to content

DRYD-2102: Fixes for Thumbnail Derivative Generation#533

Open
mikejritter wants to merge 5 commits intocollectionspace:upgrade/dryd-1633-core-updatesfrom
mikejritter:upgrade/dryd-2102-thumbnails
Open

DRYD-2102: Fixes for Thumbnail Derivative Generation#533
mikejritter wants to merge 5 commits intocollectionspace:upgrade/dryd-1633-core-updatesfrom
mikejritter:upgrade/dryd-2102-thumbnails

Conversation

@mikejritter
Copy link
Copy Markdown
Contributor

@mikejritter mikejritter commented May 7, 2026

What does this do?

  • Replace deprecated FileManager APIs
  • Add configuration for the Nuxeo Stream component
  • Add configuration for the default empty image location
  • Add default empty image
  • Misc cleanup

Why are we doing this? (with JIRA link)
Jira: https://collectionspace.atlassian.net/browse/DRYD-2102

When upgrading Nuxeo, changes were made to how their derivative generation works. This adds the necessary configuration so that the EventListener's which generate derivatives can run and submit the work items successfully.

logmanager-config.xml adds configuration for the Nuxeo Stream log. This currently uses the in memory implementation to avoid needing to deploy Kafka.

configuration-service-config.xml and the empty.png are required for the Event Listener which populates the views (derivatives) for an image. We could potentially add our own event listener which computes the views in order to avoid the requirement of the empty picture view property.

How should this be tested? Do these changes have associated tests?

  • Rebuild and start CollectionSpace
  • Upload a new MediaHandling record with an attached blob
  • Refresh the page and see that the derivatives were generated
    • Note: disabling the cache might be necessary so that the server is queried for the updated images.

Dependencies for merging? Releasing to production?
Need to discuss with hosting about if we want to deploy Kafka or keep the in memory log manager.

Has the application documentation been updated for these changes?
No.

Did someone actually run this code to verify it works?
@mikejritter tested locally. It might be good to double check the blob integration tests as well to verify the generated images are correct.

@mikejritter mikejritter force-pushed the upgrade/dryd-2102-thumbnails branch from f9114ad to 1c9913b Compare May 8, 2026 21:17
@mikejritter mikejritter marked this pull request as ready for review May 8, 2026 21:47
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