eHoldings app unusable in Honeysuckle official release
Description
CSP Request Details
CSP Rejection Details
Potential Workaround
Attachments
- 16 Dec 2020, 01:53 PM
- 16 Dec 2020, 01:53 PM
- 16 Dec 2020, 05:31 AM
- 09 Dec 2020, 02:02 AM
- 07 Dec 2020, 10:20 AM
- 06 Dec 2020, 09:40 PM
- 06 Dec 2020, 09:40 PM
- 04 Dec 2020, 03:18 PM
- 04 Dec 2020, 12:58 PM
- 04 Dec 2020, 12:58 PM
defines
relates to
Checklist
hideTestRail: Results
Activity
Pavlo Smahin December 17, 2020 at 10:20 AM
@Sobha Duvvuri @Khalilah Gambrell @Oleksii Petrenko, I created a bug related to unexpected work of status endpoint. The remaining work will be done in the https://folio-org.atlassian.net/browse/MODKBEKBJ-550#icft=MODKBEKBJ-550.
Sobha Duvvuri December 16, 2020 at 1:53 PM
@Pavlo Smahin: My observation is that UI always requested the status from backend. In Goldenrod, backend would return a 200 with "isConfiguration" : false where as in Honeysuckle - backend is returning a 500. Please see screenshots below.
Denys Bohdan December 16, 2020 at 1:51 PMEdited
@Pavlo Smahin status endpoint shouldn't be used in settings. From the code it looks like we aren't showing any errors if the request fails.. but it seems that we actually are showing it. Need to investigate it a bit more
cc: @Sobha Duvvuri
Pavlo Smahin December 16, 2020 at 7:32 AM
@Sobha Duvvuri
It is expected behavior for GET /status endpoint when there are no valid credentials.
It should be possible to update the dummy credentials from UI. We should ask FE team why they use status endpoint in settings.
cc: @Denys Bohdan
Sobha Duvvuri December 16, 2020 at 5:31 AMEdited
@Pavlo Smahin: I deployed v3.6.4 to our environment and I am able to confirm that the DB schema error(different schemas for different tenants) got fixed. However, the app is still unusable from the UI because the GET /status endpoint fails with a 500 - Rest of the information in the ticket description is still valid. Attached are the logs.
I am able to workaround the problem by hitting the api directly by making a PUT request to
/eholdings/kb-credentials/80898dee-449f-44dd-9c8e-37d5eb469b1d
to replace dummy creds with real ones and then eHoldings app works ok. But, we can't configure it from the UI.
Deployed Honeysuckle GA release versions on to an environment - ui-eholdings 5.0.2 and mod-kb-ebsco-java-3.6.3. Note: This tenant is loaded with sample and reference data.
When trying to configure eHoldings or clicking on the app, we see an error as in the screenshot attached, making the app pretty much unusable.
Attached are the logs - eHoldings status endpoint fails with a 500