Hi Lars Vornholt,
I am looking at the master branch right now and it seems the actual service classes (that define the web endpoints) for the file storage service are missing. Do you have com.jivesoftware.addon.example.storageprovider.file.services.HealthService or FileStorageService or AttachmentStorageService.java files in your version of the code?
Also the definition.template does not contain the storageDefinition but I guess you must have solved that to be seeing the health check.
Could you confirm if you have these classes, I will look into adding these classes to give the complete example.
Hi David Nicholls,
yes I add the missing storageDefinition to the definition.template.
and we also have all three mentioned classes. Only the package structure is a bit different:
thanks for the quick response
I have just committed changes to the Java SDK Example so that the file storage provider is working as an example out of the box.
- Registering MultiPartFeature in MyExampleAddOn class (to support file upload)
- Scanning com.jivesoftware.addon.example.storage.file.services in MyExampleAddOn so web service end points are picked up.
- Added 5 service classes to com.jivesoftware.addon.example.storage.file.services - these are the end points to handle the ESP end points.
- Added the definition for the storageDefinition to definition.template
This should now work but please let me know if you have any problems. This storage provider supports uploaded files and attachments and stores these files in jive-addon/filestore folder under the place identifier (number) that you configure when you register a place. The unique ids are managed in property files that get updated in jive-addon/external-id folder.
Hope this helps
I see now, these services were not where I was expecting them to be. I do not have permission after all to update the master branch and so have put them in my fork on github. You can view these here in the meantime :davidnicholls1981/jive-sdk-java-jersey · GitHub
I will ask Ryan Rutan, if he could pull these changes in, including moving where the file storage services are located as I have tested they work with the new service location com.jivesoftware.addon.example.storage.file.services
Apologies for the confusion but if you are in a hurry with this please take a look at the changes in my forked version of the code.
Can you do a pull request against the repo and I'll approve today?
Many thanks for your assistance