SCM URL normalization mishandles query strings, fragments and scp-like URLs
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 78/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- java
- Domain
- build-system
Research direction
Start at ModelInheritanceAssembler.appendPath, named in the issue, and inspect how it normalizes SCM URLs. Check the cases where .. appears in a query or fragment and where the URL is scp-like; the issue gives before-and-after examples. Done means both forms retain the intended URL components, with behavior covered by focused tests.
Written by the indexing model from the issue text.
Description
ModelInheritanceAssembler.appendPath normalizes everything after :// as a path, which mishandles two kinds of SCM URL:
- A
..in a query string or fragment is resolved against the path:http://host/repo?p=a/../bbecomeshttp://host/b. - An scp-like URL has no
://, so its scheme and host become path segments:scm:git:[email protected]:org/../repo.gitbecomesrepo.git.
Both behave the same before and after #334, which fixed the other normalization cases from #274. Cutting the query and fragment off before normalizing, and leaving host:path forms alone, would cover them.
- Dominant language
- Java
- Stars
- 17
- Forks
- 20
- Avg merge
- 12h 3m
- Merged PRs (30d)
- 22
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- No contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from apache/maven-remote-resources-plugin
-
bug
Difficulty 4/5 3-5 days Newbie friendliness 52/100
apache/maven-remote-resources-plugin#341 ·
Maintainers usually reply within 1 day
-
enhancement
Difficulty 4/5 3-5 days Newbie friendliness 48/100
apache/maven-remote-resources-plugin#324 ·
Maintainers usually reply within 1 day
-
enhancement
Difficulty 4/5 3-5 days Newbie friendliness 48/100
apache/maven-remote-resources-plugin#322 ·
Maintainers usually reply within 1 day
-
Perf: triggers a Maven model build for every dependency artifactPossibly taken A pull request linked to this issue is open or already merged. Openenhancement
Difficulty 4/5 3-5 days Newbie friendliness 55/100
apache/maven-remote-resources-plugin#270 ·
Maintainers usually reply within 1 day
-
[MRRESOURCES-106] ProcessRemoteResourcesMojo#getProjects() can end up consuming a lot of memory for big projectsMay be free again @hboutemy claimed this 496 days ago, and no pull request is open. Openbug priority:major
apache/maven-remote-resources-plugin#233 · 8 comments · 1 assignee ·
Maintainers usually reply within 1 day
All issues in apache/maven-remote-resources-plugin
Similar issues
-
Component - Build
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 77/100
navikt/mock-oauth2-server#1149 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
oracle/graalvm-reachability-metadata#11318 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Stirling-Tools/Stirling-PDF#8709 · 1 reaction ·
Maintainers usually reply within 1 day
-
area/config
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
quarkusio/quarkus#57910 · 1 comment ·
Maintainers usually reply within 1 day