feat: improve sanitization of AZKV keys

[paullatzelsperger]

What this PR changes/adds

Improves the sanitization of Azure Keyvault secret names.

Why it does that

Make the AZKV more fault-tolerant, avoid avoidable exceptions.

Further notes

• does not truncate the string to 127 characters, as that could potentially cause name collisions. In case of a length violation, the AZKV would report an error.

Linked Issue(s)

Closes #14

[github-actions]

We are always happy to welcome new contributors ❤️ To make things easier for everyone, please make sure to follow our contribution guidelines, check if you have already signed the ECA, and relate this pull request to an existing issue or discussion.

[github-actions]

Test Results

  57 files  ±  0    57 suites  ±0   22m 16s ⏱️ - 6m 32s
550 tests +10  541 ✔️ +10  9 💤 ±0  0 ❌ ±0 
560 runs  +10  551 ✔️ +10  9 💤 ±0  0 ❌ ±0 

Results for commit 085bfdf. ± Comparison against base commit 4d7aff2.

This pull request removes 1 and adds 11 tests. Note that renamed tests count towards both.

org.eclipse.edc.connector.dataplane.azure.datafactory.AzureDataFactoryTransferManagerTest ‑ [5] [Queued, InProgress, 1767afb6-d4fc-49f0-9e5e-ef9832251e7b]

org.eclipse.edc.connector.dataplane.azure.datafactory.AzureDataFactoryTransferManagerTest ‑ [5] [Queued, InProgress, dd93b1c2-4161-4c29-bfd2-dfd07ffea569]
org.eclipse.edc.vault.azure.AzureVaultTest ‑ [1] foo_bar
org.eclipse.edc.vault.azure.AzureVaultTest ‑ [2] foo%bar
org.eclipse.edc.vault.azure.AzureVaultTest ‑ [3] foo.bar
org.eclipse.edc.vault.azure.AzureVaultTest ‑ [4] foo^bar
org.eclipse.edc.vault.azure.AzureVaultTest ‑ [5] foo#bar
org.eclipse.edc.vault.azure.AzureVaultTest ‑ [6] foo;bar
org.eclipse.edc.vault.azure.AzureVaultTest ‑ [7] foo_&*_bar
org.eclipse.edc.vault.azure.AzureVaultTest ‑ [8] foo__bar
org.eclipse.edc.vault.azure.AzureVaultTest ‑ [9] foo%%@#$bar
…

♻️ This comment has been updated with latest results.