You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@felix.apache.org by GitBox <gi...@apache.org> on 2021/05/15 21:47:32 UTC
[GitHub] [felix-dev] PiotrKlimczak opened a new pull request #76: FELIX-6416 Fix url handling
PiotrKlimczak opened a new pull request #76:
URL: https://github.com/apache/felix-dev/pull/76
Not sure if is safe to have just url.getPath() as I am not Felix Framework expert.
But even if this PR is not suitable for merge, at least it shows how special characters should be handled and also it has test coverage for the scenario encountered.
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [felix-dev] karlpauls closed pull request #76: FELIX-6416 Fix url handling
Posted by GitBox <gi...@apache.org>.
karlpauls closed pull request #76:
URL: https://github.com/apache/felix-dev/pull/76
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [felix-dev] karlpauls commented on pull request #76: FELIX-6416 Fix url handling
Posted by GitBox <gi...@apache.org>.
karlpauls commented on pull request #76:
URL: https://github.com/apache/felix-dev/pull/76#issuecomment-848820461
Closing in favor of #77 - many thanks for the test case!
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org