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

AppBox error - 'crc check failed for file' #201

Closed
vikas-chandra-mnnit opened this issue Sep 24, 2020 · 11 comments
Closed

AppBox error - 'crc check failed for file' #201

vikas-chandra-mnnit opened this issue Sep 24, 2020 · 11 comments

Comments

@vikas-chandra-mnnit
Copy link

After upgrading the macOS to Catalina I have started facing this issue. I am using AppBox version Version 2.9.1 (1).

@Gahlot
Copy link

Gahlot commented Sep 26, 2020

When try to upload a .ipa made by Xcode(12) getting this error. works fine for .ipa made by XCode(11.3).

@vineetchoudhary
Copy link
Member

@vikas-chandra-mnnit I'm using AppBox on Catalina since last year and didn't face this issue. @Gahlot ipa file generated by Xcode 12 is also working fine.

Can you please provide my ipa file in which you are facing this issue? You can also email me on vineetchoudhary[at]live.in

@vikas-chandra-mnnit
Copy link
Author

@vineetchoudhary Thanks for the reply. I am attaching the IPA with this comment. This ipa is generated on Xcode 12.
ESP Schedule.ipa.zip

@vineetchoudhary
Copy link
Member

@vikas-chandra-mnnit I'm able to upload this IPA file without any error. I'm also using AppBox version 2.9.1(1) on macOS Catalina 10.15.7.

https://www.youtube.com/watch?v=1zTmrjjY-ts

@vikas-chandra-mnnit
Copy link
Author

vikas-chandra-mnnit commented Sep 28, 2020

@vineetchoudhary I am also using the same version of AppBox and macOS Catalina 10.15.6. Please tell me what other dependency AppBox uses internally that might cause the error.

@vineetchoudhary
Copy link
Member

AppBox internally uses ZipArchive to extract the IPA file. They are also not able to reproduce this issue.

@kreshikhin
Copy link

I met this error too when I used proxy which corrupts some https requests. So checking of proxy or network settings may be helpful for resolving this issue.

@tkirby
Copy link
Contributor

tkirby commented Mar 16, 2022

Getting this error on a few builds lately. Seems like some issue with the zip library. I've checked the .ipa and it unzips fine with no errors.

2022-03-17 12:07:15.504 AppBox[45051:3886714] [SSZipArchive] Error: The file “AddAttractionsInfoView.nib” couldn’t be saved in the folder “Blackbook Travels” because a file with the same name already exists.
2022-03-17 12:07:15.504 AppBox[45051:3886698] Extracting files...
2022-03-17 12:07:15.504 AppBox[45051:3886698] 723-3218-Payload/Blackbook Travels.app/AddAttractionsInfoView.nib/
2022-03-17 12:07:15.647 AppBox[45051:3886714] [SSZipArchive] Error: The file “AddAttractionsInfoView.nib” couldn’t be saved in the folder “Blackbook Travels” because a file with the same name already exists.
2022-03-17 12:07:15.647 AppBox[45051:3886714] [SSZipArchive] Failed to open file on unzipping.(Error Domain=NSPOSIXErrorDomain Code=20 "Not a directory")
2022-03-17 12:07:15.647 AppBox[45051:3886698] ALERT -
Title - AppBox - Error Message - crc check failed for file

Might be cleaner if to only unzip the plist and manifest files needed and left the rest of the .ipa alone to avoid this issue.

@tkirby
Copy link
Contributor

tkirby commented Mar 17, 2022

I did some more digging on this and I removed the file that AppBox is complaining already exists (in temp directory AppBox is unzipping to) and after rerunning and adding the same .ipa, it worked. Seems like AppBox is extracting to the same place every time and sometimes having it fail due to file already esists.

@tkirby
Copy link
Contributor

tkirby commented Mar 17, 2022

OKay I fixed this by just createing a unique directory in the temp directory to unzip to. So instead of unzip to...

/var/folders/pl/_72l1lfs59n4tqh0nby6zl7c0000gn/T/

AppBox will add a unque directory in the temp dir...

/var/folders/pl/_72l1lfs59n4tqh0nby6zl7c0000gn/T/41A9EF55-B5C5-44D4-B5B6-E4C59BDD0EE8

That way there is no chance of a file collision during unzip.

tkirby pushed a commit to tkirby/AppBox-iOSAppsWirelessInstallation that referenced this issue Mar 17, 2022
Create unique temp directory when unzipping files to avoid potential collisions.

getappbox#201
@vineetchoudhary
Copy link
Member

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

No branches or pull requests

5 participants