Wouter Born

Results 410 comments of Wouter Born

That path seems to exceed the max path length of Windows that is 260 chars according to these [docs](https://docs.microsoft.com/en-us/windows/win32/fileio/naming-a-file#maximum-path-length-limitation). There are 264 chars in: `C:\Users\Familjen i Lerkil\AppData\Local\Temp\openhab\backup-2.4.0\runtime\system\org\eclipse\smarthome\automation\org.eclipse.smarthome.automation.module.script.defaultscope\0.10.0.oh240\org.eclipse.smarthome.automation.module.script.defaultscope-0.10.0.oh240.jar` Perhaps [enabling longer...

Did you already investigate this issue by monitoring the subscriber count and cleanup by enabling debug logging on `org.openhab.core.io.rest.sitemap`? It will only return this error when the limit of 50...

I did have some time now to look into this and after enabling the debug logging on `org.openhab.core.io.rest.sitemap` it looks like the openHAB Android App is creating a new SSE...

It would be interesting to know if this issue also occurs with 2.5.x. If so it would most likely be an issue in the App. In OH3 there is a...

I agree that using a pool would be a nice approach to limit the number of subscriptions. How many pages does it keep open in the background? There could still...

> How can a client release subscriptions? I asked earlier already because I don't see a way of doing that. When the client closes the connection this should be detected...

Yes the pooling will certainly help with reducing the number of active subscriptions as will fixing any cleanup bugs and perhaps it should also run the cleanup more often and...

After some debugging and fixes I think the cleanup works a lot better with https://github.com/openhab/openhab-core/pull/1679.

The [SitemapResource history](https://github.com/openhab/openhab-core/commits/main/bundles/org.openhab.core.io.rest.sitemap/src/main/java/org/openhab/core/io/rest/sitemap/internal/SitemapResource.java) doesn't suggest an unsubscribe operation was added. So openhab/openhab-core#1679 fixed/prevented most of the issues users have.

It's probably nice to have now. Just like having an unsubscribe method. :wink: