fix(request-parameter): deduplicate parameters using ConcurrentDictionary

[hamzamahmood]

What

This PR replaces the internal parameter collection inside Parameter.Builder with a ConcurrentDictionary<string, Parameter>, using each parameter’s key as the dictionary key.

Why

Previously, we used a ConcurrentBag to collect parameters, which:

• Caused duplicates when multiple parameters had the same key.
• Introduced ordering issues.
• Allowed stale values (e.g., expired tokens) to be reused unintentionally.

The new implementation:

• Ensures only one entry per parameter key (latest value wins).
• Resolves long-standing parameter accumulation issues across requests.
• Aligns with expected AuthManager behavior (i.e., always using the most recent token).
• Maintains thread safety while improving correctness.

Type of change

Select multiple if applicable.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause a breaking change)
• Tests (adds or updates tests)
• Documentation (adds or updates documentation)
• Refactor (style improvements, performance improvements, code refactoring)
• Revert (reverts a commit)
• CI/Build (adds or updates a script, change in external dependencies)

Checklist

• My code follows the coding conventions
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added new unit tests

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud