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

[fastlane_core] escape xcrun itmstransporter password for special characters #16795

Merged

Conversation

joshdholtz
Copy link
Member

Motivation and Context

Fixes #16777

Description

Copy link
Member

@janpio janpio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ui, interesting!

Did you run and test this locally? I do not have the appropriate Mac right now, but if you saw this work it is good to merge.

@joshdholtz
Copy link
Member Author

Yup yup! I was able to replicate the issue and fix. I spent a stupid amount of hours trying to figure this out 😛

@joshdholtz joshdholtz merged commit 74da17d into master Jul 8, 2020
@joshdholtz joshdholtz deleted the joshdholtz-shell-escape-xcrun-itmstransporter-password branch July 8, 2020 20:01
This was referenced Jul 9, 2020
Copy link

@fastlane-bot fastlane-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Congratulations! 🎉 This was released as part of fastlane 2.151.2 🚀

@janpio
Copy link
Member

janpio commented Jul 9, 2020

Tbh I do not fully understand why this works better. Probably some shell interpolating stuff?

minuscorp pushed a commit to minuscorp/fastlane that referenced this pull request Jul 18, 2020
@fastlane fastlane locked and limited conversation to collaborators Sep 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deliver errors when password contains a dollar sign $
3 participants