Configure path prefix for the API server

[tnajdek]

(I've split this from another issue, as it pertains to configuring the API server and not the web library's routing.)

Dear @glorieux-f and @tnajdek,

I run into a similar problem: the current zotero-webapp implementation allows to configure the apiAuthorityPart in index.html, like below. But how is it possible to change the API path of the API server ? So, e.g., in the case of @glorieux-f ,
how would I set https://www.unige.ch/piaget/api/ as the route to the zotero API endpoint ?
(We do not want to set up a second domain, just for the zotero server).

Thanks and merci !

<script type="application/json" id="zotero-web-library-config">
		{
			"userSlug": "zotero-user",
			"userId": "475425",
			"apiConfig": {
				"apiAuthorityPart": "api.zotero.org"
			},
			"libraries": {
				"include": [
					{
						"key": "g729",
						"name": "All Things Zotero",
						"isGroupLibrary": true
					}
				],
				"includeMyLibrary": true,
				"includeUserGroups": false
			}
		}
	</script>

Originally posted by @markdoerr in #534 (comment)

[markdoerr]

@tnajdek,
thanks for opening a new issue 👍 - I hope it can be easily solved :)

[tnajdek]

So basically apiConfig is passed to zotero-api-client and eventually used here and here.

It should be trivial to add support for apiPathPrefix or something along these lines.

[markdoerr]

Thanks, @tnajdek, for pointing at the positions in the code. Yes, indeed it would be trivial to add this extra option here, like, e.g. :

const url = https://${options.apiAuthorityPart}/${path}${query};

-> const url = https://${options.apiAuthorityPart}/${options.apiPathPrefix}${query};

Will you do this please ? (I am heading for holidays tomorrow, so I would only have time to make a PR in September :( )

[markdoerr]

or better (?) :
const url = https://${options.apiAuthorityPart}/${options.apiPathPrefix}${path}${query};

[markdoerr]

Thanx, @tnajdek,
for adding this small, but important feature :) - I will test it in the next days ...

[tnajdek]

Please note that this feature is currently only available in the underlying zotero-api-client dependency and is not yet included in the web library. To use it, you will need to update the dependency locally in your build. I will update the web library before the next release.

[markdoerr]

Thanks for the hint, Tom (I guessed so ;) - looking also forward to the next release....

[tnajdek]

Resolved in 2553365