fix #281

[inf17101]

fix #281
changes regex to allow additional customized ids instead of ids in uuid format only.

Description

Better would be /(?:[0-9a-z]+-[0-9a-z]+)+?/ as regex expression because of non-capturing group.
Or to allow only a certain range of characters: /(?:/)([0-9a-z-]{1,100})?/

Issues Resolved

[List any issues this PR will resolve]

Check List

• New functionality includes testing.
  • All tests pass, including unit test, integration test and doctest
• New functionality has been documented.
  • New functionality has javadoc added
  • New functionality has user manual doc added
• Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[codecov-commenter]

Codecov Report

Merging #289 (2c9271e) into main (3b03b70) will increase coverage by 4.41%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #289      +/-   ##
============================================
+ Coverage     72.70%   77.11%   +4.41%     
============================================
  Files            94       30      -64     
  Lines          3924     1866    -2058     
  Branches        608      374     -234     
============================================
- Hits           2853     1439    -1414     
+ Misses          895      424     -471     
+ Partials        176        3     -173     

Flag	Coverage Δ
dashboards-reports	77.11% <ø> (ø)
reports-scheduler	?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...uler/model/ReportDefinitionDetailsSearchResults.kt
...n/org/opensearch/reportsscheduler/model/RestTag.kt
...pensearch/reportsscheduler/model/ReportInstance.kt
...scheduler/resthandler/OnDemandReportRestHandler.kt
...rtsscheduler/model/InContextReportCreateRequest.kt
...ortsscheduler/model/ReportInstanceSearchResults.kt
...tsscheduler/action/DeleteReportDefinitionAction.kt
...sscheduler/model/UpdateReportDefinitionResponse.kt
...nsearch/reportsscheduler/model/ReportDefinition.kt
...tsscheduler/scheduler/ReportDefinitionJobRunner.kt
... and 54 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3b03b70...2c9271e. Read the comment docs.

[zhongnansu]

I am OK with the change. But for us to merge this PR, you need to do the following

• change your PR title to something descriptive
• sign your commit with DCO, so github workflow will pass. you can refer to https://www.secondstate.io/articles/dco/

[zhongnansu]

We want to get this out in 1.3. If the contributor doesn't reply to comments and update PR by March 7th. The team will clost the PR and post another one with correct signature.