You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When trying to understand zero byte file upload issues in posixfs @aduffeck and I noticed that the places that emit events have started to fragment:
the storage provider is hardcoded to have async uploads = false, which prevents it from emitting any event when a zero byte file is uploaded
the data provider emits postprocessing related events for uploads
the eventsmiddleware interceptor emits events for metadata operations, but it has no way of emitting the path when it receives ids, something we need to properly show the end user the source and target path in move activities
@2403905 started moving some events into the storageprovider to emit events that include the path
The storage driver currently usese a config produced by OcisNoEvents(cfg) whereas the dataprovider config is produced by Ocis(cfg). Ocis(cfg) has
OcisNoEvents(cfg) omits that in order to force the finalization of zero byte file uploads. This is all very confusing and hard to reason about.
IMO we should move the eventsmiddleware into decomposedfs and evolve decomposedfs to have middlewares for resource lookup, permissioncheck and events. it should replace the reva Storage interface and become a framework for building storage drivers because it implements so many necessities we implicitly rely on.
The text was updated successfully, but these errors were encountered:
When trying to understand zero byte file upload issues in posixfs @aduffeck and I noticed that the places that emit events have started to fragment:
The storage driver currently usese a config produced by
OcisNoEvents(cfg)
whereas the dataprovider config is produced byOcis(cfg)
.Ocis(cfg)
hasOcisNoEvents(cfg)
omits that in order to force the finalization of zero byte file uploads. This is all very confusing and hard to reason about.IMO we should move the eventsmiddleware into decomposedfs and evolve decomposedfs to have middlewares for resource lookup, permissioncheck and events. it should replace the reva
Storage
interface and become a framework for building storage drivers because it implements so many necessities we implicitly rely on.The text was updated successfully, but these errors were encountered: