apache / apache/maven-resolver

[MRESOLVER-126] Support property type conversion from String into Map

Open
#859 5 comments 0 reactions 0 assignees View on GitHub
enhancement priority:major
Dominant language
Java
Stars
152
Forks
160
Avg merge
1d 4h
Merged PRs (30d)
49

Description

**[Konrad Windszus](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=kwin)** opened **[MRESOLVER-126](https://issues.apache.org/jira/browse/MRESOLVER-126?redirect=false)** and commented

The property `aether.connector.http.headers` requires a `Map` as value (https://github.com/apache/maven-resolver/blob/47edcfe69c4e52ced4cb93d65b7348b5645cdd68/maven-resolver-transport-http/src/main/java/org/eclipse/aether/transport/http/HttpTransporter.java#L139). As in https://github.com/apache/maven-resolver/blob/47edcfe69c4e52ced4cb93d65b7348b5645cdd68/maven-resolver-util/src/main/java/org/eclipse/aether/util/ConfigUtils.java#L362 there is no type conversion, it is currently impossible to configure this from Apache Maven (as Maven is using system properties).

`List` suffers from the same issue but is currently not used (https://github.com/apache/maven-resolver/blob/47edcfe69c4e52ced4cb93d65b7348b5645cdd68/maven-resolver-util/src/main/java/org/eclipse/aether/util/ConfigUtils.java#L319)

---

**Remote Links:**
- [GitHub Pull Request #63
](https://github.com/apache/maven-resolver/pull/63)

Contributor guide

No contributing guide indexed for this repository

Research direction

Start in maven-resolver-util/src/main/java/org/eclipse/aether/util/ConfigUtils.java around lines 319 and 362, then inspect maven-resolver-transport-http/src/main/java/org/eclipse/aether/transport/http/HttpTransporter.java around line 139. Review pull request #63 and the existing tests; done means system properties can configure the Map value, with List handling addressed as described.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
build-system
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.