Re: [PR] [MRESOLVER-540] TransferResource can reveal more about resource [maven-resolver]

2024-04-18 Thread via GitHub
cstamas merged PR #474: URL: https://github.com/apache/maven-resolver/pull/474 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] [MRESOLVER-540] TransferResource can reveal more about resource [maven-resolver]

2024-04-18 Thread via GitHub
cstamas commented on PR #474: URL: https://github.com/apache/maven-resolver/pull/474#issuecomment-2064602041 Simplified, removed cruft and is not simple and nice. @gnodet pls review -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [MRESOLVER-540] TransferResource can reveal more about resource [maven-resolver]

2024-04-18 Thread via GitHub
cstamas commented on PR #474: URL: https://github.com/apache/maven-resolver/pull/474#issuecomment-2064257759 Will remove/simplify it -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] [MRESOLVER-540] TransferResource can reveal more about resource [maven-resolver]

2024-04-18 Thread via GitHub
cstamas commented on code in PR #474: URL: https://github.com/apache/maven-resolver/pull/474#discussion_r1570983720 ## maven-resolver-api/src/main/java/org/eclipse/aether/transfer/TransferResource.java: ## @@ -23,17 +23,43 @@ import org.eclipse.aether.RequestTrace; +import

Re: [PR] [MRESOLVER-540] TransferResource can reveal more about resource [maven-resolver]

2024-04-18 Thread via GitHub
gnodet commented on code in PR #474: URL: https://github.com/apache/maven-resolver/pull/474#discussion_r1570963455 ## maven-resolver-api/src/main/java/org/eclipse/aether/transfer/TransferResource.java: ## @@ -23,17 +23,43 @@ import org.eclipse.aether.RequestTrace; +import

Re: [PR] [MRESOLVER-540] TransferResource can reveal more about resource [maven-resolver]

2024-04-18 Thread via GitHub
cstamas commented on PR #474: URL: https://github.com/apache/maven-resolver/pull/474#issuecomment-2063761537 Main inspiration is to make stunts like these not required: https://github.com/apache/maven-mvnd/pull/930 -- This is an automated message from the Apache Git Service. To respond