diff --git a/dashboards-reports/public/components/report_definitions/report_settings/report_settings.tsx b/dashboards-reports/public/components/report_definitions/report_settings/report_settings.tsx index 2ecf927a..3317bb5a 100644 --- a/dashboards-reports/public/components/report_definitions/report_settings/report_settings.tsx +++ b/dashboards-reports/public/components/report_definitions/report_settings/report_settings.tsx @@ -658,7 +658,11 @@ export function ReportSettings(props: ReportSettingProps) { }); await httpClientProps - .get('../api/notebooks/') + .get('../api/observability/notebooks/') + .catch((error: any) => { + console.error('error fetching notebooks, retrying with legacy api', error) + return httpClientProps.get('../api/notebooks/') + }) .then(async (response: any) => { let notebooksOptions = getNotebooksOptions(response.data); reportSourceOptions.notebooks = notebooksOptions; diff --git a/dashboards-reports/server/utils/__tests__/validationHelper.test.ts b/dashboards-reports/server/utils/__tests__/validationHelper.test.ts index 41ce3b49..cacd3d61 100644 --- a/dashboards-reports/server/utils/__tests__/validationHelper.test.ts +++ b/dashboards-reports/server/utils/__tests__/validationHelper.test.ts @@ -65,7 +65,8 @@ const createReportInput: ReportSchemaType = { report_definition: createReportDefinitionInput, }; -const createReportDefinitionNotebookInput: ReportDefinitionSchemaType = { +// this is the url format used before notebooks merged into observability +const createReportDefinitionNotebookLegacyInput: ReportDefinitionSchemaType = { report_params: { report_name: 'test notebooks report', report_source: REPORT_TYPE.notebook, @@ -90,6 +91,31 @@ const createReportDefinitionNotebookInput: ReportDefinitionSchemaType = { }, } +const createReportDefinitionNotebookInput: ReportDefinitionSchemaType = { + report_params: { + report_name: 'test notebooks report', + report_source: REPORT_TYPE.notebook, + description: 'Hi this is your Notebook on demand', + core_params: { + base_url: `/app/observability#/notebooks/${SAMPLE_SAVED_OBJECT_ID}`, + window_width: 1300, + window_height: 900, + report_format: FORMAT.pdf, + time_duration: 'PT5M', + origin: 'http://localhost:5601', + }, + }, + delivery: { + configIds: [], + title: 'title', + textDescription: 'text description', + htmlDescription: 'html description' + }, + trigger: { + trigger_type: TRIGGER_TYPE.onDemand, + }, +} + describe('test input validation', () => { test('create report with correct saved object id', async () => { const savedObjectIds = [`dashboard:${SAMPLE_SAVED_OBJECT_ID}`]; @@ -118,6 +144,16 @@ describe('test input validation', () => { expect(report).toBeDefined(); }); + test('create notebook report definition with legacy base url format', async () => { + const savedObjectIds = [`notebook:${SAMPLE_SAVED_OBJECT_ID}`]; + const client = mockOpenSearchClient(savedObjectIds); + const report = await validateReportDefinition( + client, + createReportDefinitionNotebookLegacyInput + ); + expect(report).toBeDefined(); + }); + test('create notebook report definition with correct base url format', async () => { const savedObjectIds = [`notebook:${SAMPLE_SAVED_OBJECT_ID}`]; const client = mockOpenSearchClient(savedObjectIds); diff --git a/dashboards-reports/server/utils/validationHelper.ts b/dashboards-reports/server/utils/validationHelper.ts index 705c8cd0..590e1e9a 100644 --- a/dashboards-reports/server/utils/validationHelper.ts +++ b/dashboards-reports/server/utils/validationHelper.ts @@ -37,7 +37,10 @@ import { REPORT_TYPE } from '../../server/routes/utils/constants'; export const isValidRelativeUrl = (relativeUrl: string) => { let normalizedRelativeUrl = relativeUrl - if (!relativeUrl.includes('notebooks-dashboards')) { + if ( + !relativeUrl.includes('observability#/notebooks') && + !relativeUrl.includes('notebooks-dashboards') + ) { normalizedRelativeUrl = path.posix.normalize(relativeUrl); } @@ -55,7 +58,7 @@ export const isValidRelativeUrl = (relativeUrl: string) => { export const regexDuration = /^(-?)P(?=\d|T\d)(?:(\d+)Y)?(?:(\d+)M)?(?:(\d+)([DW]))?(?:T(?:(\d+)H)?(?:(\d+)M)?(?:(\d+(?:\.\d+)?)S)?)?$/; export const regexEmailAddress = /\S+@\S+\.\S+/; export const regexReportName = /^[\w\-\s\(\)\[\]\,\_\-+]+$/; -export const regexRelativeUrl = /^\/(_plugin\/kibana\/|_dashboards\/)?app\/(dashboards|visualize|discover|notebooks-dashboards\?view=output_only)([?&]security_tenant=.+|)#\/(view\/|edit\/)?[^\/]+$/; +export const regexRelativeUrl = /^\/(_plugin\/kibana\/|_dashboards\/)?app\/(dashboards|visualize|discover|observability|notebooks-dashboards\?view=output_only)([?&]security_tenant=.+|)#\/(notebooks\/|view\/|edit\/)?[^\/]+$/; export const validateReport = async ( client: ILegacyScopedClusterClient, diff --git a/dashboards-reports/test/httpMockClient.js b/dashboards-reports/test/httpMockClient.js index 084f1b85..40efdc2a 100644 --- a/dashboards-reports/test/httpMockClient.js +++ b/dashboards-reports/test/httpMockClient.js @@ -38,6 +38,12 @@ httpClientMock.get = jest.fn(() => ({ })), catch: jest.fn(), })), + catch: jest.fn(() => ({ + then: jest.fn(() => ({ + catch: jest.fn() + })), + catch: jest.fn(), + })), })); httpClientMock.head = jest.fn(); httpClientMock.post = jest.fn(() => ({