Fix crash caused by use of API from older Multipart versions

[rogerluan]

This error was happening when attempting to run wti push <filepath>:

This PR fixes this.

[edouard]

@rogerluan Thanks for the PR! I'll review it today. Did you encounter this issue when using fastlane with multipart_post locked to version 2.0?

[rogerluan]

@edouard Yes: multipart-post (2.0.0) and fastlane (2.212.1)

[rogerluan]

Which just reminded me... my changes might not work for those using multipart <2.0 😅

I think we could either enforce multipart to be >=2.0, or we'll have to add runtime checks to see which API is available 😬

[edouard]

Which just reminded me... my changes might not work for those using multipart <2.0 😅

I think we could either enforce multipart to be >=2.0, or we'll have to add runtime checks to see which API is available 😬

I think it's fine to require multipart_post >= 2.0. The 2.0 version of multipart_post was released in 2013 so it's not like the cutting edge version of that library 😅

I wish @fastlane would relax that dependency on multipart_post though, I’m not sure why it’s locked to a 10 years old version of that library. I've created a pull request fastlane/fastlane#20870 but it hasn't been merged yet.

[rogerluan]

I've reviewed your PR in fastlane repo and approved it @edouard 🙏 please take a look at it again 😊

Thanks for getting this merged nonetheless!