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

Bug fix for not updating the header when autoRedirect = false #16

Closed
wants to merge 2 commits into from

Conversation

kazuyukitanimura
Copy link
Contributor

I debugged why autoRedirect property of Ti.Network.createHTTPClient is ignored for 3.3.0.GA. The default property of autoRedirect is true.

When autoRedirect = false is set, self.response.status is always zero in APSHTTPRequest.m
Therefore the redirect check if statement
if (!self.redirects && self.response.status != 0)
does always fall through.
[self.response updateResponseParamaters:response];
needs to be called in advance

@ingo
Copy link
Contributor

ingo commented Jul 28, 2014

Thank you! @kazuyukitanimura is there a related JIRA ticket for this issue?

@kazuyukitanimura
Copy link
Contributor Author

Here is the jira https://jira.appcelerator.org/browse/TC-4483

I can revert the xcode project will do later. Just curious, why did the project got over written? What if we put it in the .gitignore? Is there a better way? How do ppl share git repo over xcode?

@jhaynie
Copy link
Contributor

jhaynie commented Jul 29, 2014

yeah, .xcworkspace should have been in .gitignore and not commited.

@kazuyukitanimura
Copy link
Contributor Author

Removed the problem file

@pec1985
Copy link
Contributor

pec1985 commented Sep 19, 2014

Closing, refer to #17

@pec1985 pec1985 closed this Sep 19, 2014
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.

None yet

4 participants