Skip to content
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

fixes error with unpacking packages #145

Merged
merged 5 commits into from
Sep 24, 2023

Conversation

tymmej
Copy link
Contributor

@tymmej tymmej commented Aug 29, 2023

sometimes first layer of package did have different normalization that item.name and script couldn't open it

also normalize content of archive after unpacking, same issue as in pull request #144

sometimes first layer of package did have different normalization that item.name and script couldn't open it
also normalize content of archive after unpacking, same issue as in pull request mandarons#144
@tymmej tymmej changed the title this fixes error with unpacking packages fixes error with unpacking packages Aug 29, 2023
@mandarons mandarons enabled auto-merge (rebase) August 29, 2023 17:28
@mandarons
Copy link
Owner

@tymmej Can you please provide a sample package file which needs to be normalized? I want to add it to test data and write a test with it.

@tymmej
Copy link
Contributor Author

tymmej commented Sep 21, 2023

Inside archive test.tar.gz is package "Fotoksiążka Wzór.xmcf" which causes problem.

If you want minimal example I believe any package with non-ASCII characters in filename causes problem - https://gist.github.com/JamesChevalier/8448512?permalink_comment_id=2846892

Some non-ASCII characters to test: ĄąÓóŻż (polish letters)

@mandarons mandarons mentioned this pull request Sep 21, 2023
@mandarons mandarons merged commit f85aa35 into mandarons:main Sep 24, 2023
1 check passed
@mandarons mandarons added this to the 1.15.0 milestone Sep 24, 2023
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