changes regex to allow additional customized saved object ids

[zhongnansu]

Signed-off-by: Zhongnan Su szhongna@amazon.com

Description

This code logic is contributed by @inf17101, in this #289. Since we need this fix to go out the upcoming release, and we decided to take the logic and commit, instead of waiting for the author to reply

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

#281

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.

[davidcui1225]

Thanks for the fix- I'll open a branch to investigate the link checker errors further and see what alternate links we can include if no other solution is found

[zhongnansu]

Thanks for the fix- I'll open a branch to investigate the link checker errors further and see what alternate links we can include if no other solution is found

Thanks David

[codecov-commenter]

Codecov Report

Merging #295 (8902e89) into main (d444010) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #295   +/-   ##
=========================================
  Coverage     72.75%   72.75%           
  Complexity      369      369           
=========================================
  Files            94       94           
  Lines          3928     3928           
  Branches        609      609           
=========================================
  Hits           2858     2858           
  Misses          894      894           
  Partials        176      176           

Flag	Coverage Δ
dashboards-reports	77.21% <ø> (ø)
reports-scheduler	68.70% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 d444010...8902e89. Read the comment docs.