overlay: add proxy debug routing and bump v0.1.8

Author: joonsoome

.env.portainer.example

@@ -1,8 +1,8 @@
 # Portainer stack environment (example values). Copy to `.env.portainer` for Portainer UI deployment.
 
 # GHCR images to deploy (choose tags built by CI) — default to a pinned release tag
-PDF_PROXY_IMAGE=ghcr.io/joonsoome/on-prem-zotero-webui/pdf-proxy:v0.1.7
-WEB_LIBRARY_IMAGE=ghcr.io/joonsoome/on-prem-zotero-webui/web-library:v0.1.7
+PDF_PROXY_IMAGE=ghcr.io/joonsoome/on-prem-zotero-webui/pdf-proxy:v0.1.8
+WEB_LIBRARY_IMAGE=ghcr.io/joonsoome/on-prem-zotero-webui/web-library:v0.1.8
 
 # Host path for Zotero WebDAV data mounted into the proxy container
 ZOTERO_ROOT_HOST_PATH=/volume1/Reference/zotero

.env.stage.example

@@ -1,8 +1,8 @@
 # Staging/NAS environment configuration (example values). Copy to `.env.stage` on the staging host.
 
 # GHCR images to deploy (choose tags built by CI) — default to a pinned release tag
-PDF_PROXY_IMAGE=ghcr.io/joonsoome/on-prem-zotero-webui/pdf-proxy:v0.1.7
-WEB_LIBRARY_IMAGE=ghcr.io/joonsoome/on-prem-zotero-webui/web-library:v0.1.7
+PDF_PROXY_IMAGE=ghcr.io/joonsoome/on-prem-zotero-webui/pdf-proxy:v0.1.8
+WEB_LIBRARY_IMAGE=ghcr.io/joonsoome/on-prem-zotero-webui/web-library:v0.1.8
 
 # Host path for Zotero WebDAV data mounted into the proxy container
 ZOTERO_ROOT_HOST_PATH=/volume1/Reference/zotero

README.md

@@ -3,7 +3,7 @@
 **Self-hosted Zotero WebUI Opensource Library + WebDAV-based PDF viewer **Avoid storage fees, keep privacy, and still enjoy a full browser-based Zotero library.
 
 ## It is PoC
-It(v0.1.7) is still fully not properly works, please keep on watching to be ready for production.
+It(v0.1.8) is still fully not properly works, please keep on watching to be ready for production.
 
 ---
 

app/web-library-overlay/src/js/actions/attachments.js

@@ -32,9 +32,17 @@ const tryGetAttachmentURL = attachmentItemKey => {
 	return async (dispatch, getState) => {
 		const state = getState();
 		const item = get(state, ['libraries', state.current.libraryKey, 'items', attachmentItemKey], null);
+		const proxyBase = state.config?.pdfProxyBaseUrl;
 
 		// If a proxy base URL is configured, bypass Storage/enclosure links and use the proxy.
-		if (state.config?.pdfProxyBaseUrl) {
+		if (proxyBase) {
+			// Debug: trace when proxy URL is requested.
+			console.log('[proxy-debug] tryGetAttachmentURL using proxy', {
+				attachmentItemKey,
+				linkMode: item?.linkMode,
+				contentType: item?.contentType,
+				proxyBase,
+			});
 			return dispatch(getAttachmentUrl(attachmentItemKey));
 		}
 
@@ -111,17 +119,29 @@ const pickBestItemAction = itemKey => {
 const pickBestAttachmentItemAction = attachmentItemKey => {
 	return async (dispatch, getState) => {
 		const state = getState();
-		const current = state.current;
-		const item = get(state, ['libraries', state.current.libraryKey, 'items', attachmentItemKey], null);
-
-		const isFile = item && item.linkMode && item.linkMode.startsWith('imported') && item[Symbol.for('links')].enclosure;
-		const isLink = item && item.linkMode && item.linkMode === 'linked_url';
+	const current = state.current;
+	const item = get(state, ['libraries', state.current.libraryKey, 'items', attachmentItemKey], null);
+	const proxyBase = state.config?.pdfProxyBaseUrl;
+
+	const isFile = item && item.linkMode && item.linkMode.startsWith('imported') && item[Symbol.for('links')].enclosure;
+	const isLink = item && item.linkMode && item.linkMode === 'linked_url';
+
+	if (isFile) {
+		if (Object.keys(READER_CONTENT_TYPES).includes(item.contentType)) {
+			// If proxy is available, prefer proxy URL to force PDF fetch even when reader path is used.
+			if (proxyBase) {
+				console.log('[proxy-debug] pickBestAttachmentItemAction -> proxy open', {
+					attachmentItemKey,
+					linkMode: item.linkMode,
+					contentType: item.contentType,
+					proxyBase,
+				});
+				return openDelayedURL(dispatch(getAttachmentUrl(attachmentItemKey)));
+			}
 
-		if (isFile) {
-			if (Object.keys(READER_CONTENT_TYPES).includes(item.contentType)) {
-				const readerPath = makePath(state.config, {
-					attachmentKey: null,
-					collection: current.collectionKey,
+			const readerPath = makePath(state.config, {
+				attachmentKey: null,
+				collection: current.collectionKey,
 					items: [attachmentItemKey],
 					library: current.libraryKey,
 					noteKey: null,