Skip to content

Fix isJarFile method for paths with spaces #14

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

tyilo
Copy link

@tyilo tyilo commented Sep 9, 2021

Without this loading bundled the bundled libsodium using lazy-sodium will fail, if the jar is in a directory containing spaces.

@gurpreet-
Copy link
Collaborator

Hi @tyilo,

Many thanks for your first contribution.

It seems this fix #12 has caused some conflicts on your PR. Can you verify if that fix fixes your bug?

@tyilo
Copy link
Author

tyilo commented Jan 3, 2022

Unfortunately I can confirm that the bug is still present in version 2.0.2.
I will update the merge request.

@tyilo
Copy link
Author

tyilo commented Jan 3, 2022

@gurpreet-
It now works again and I've also added a test.

@tyilo
Copy link
Author

tyilo commented Jul 19, 2022

@gurpreet- Can you merge this?

@tyilo
Copy link
Author

tyilo commented Nov 6, 2023

@gurpreet- Ping

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants