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

[TIMOB-24960] Android: TiHTTPClient credentials encoding issue. #9220

Merged
merged 1 commit into from Jul 14, 2017

Conversation

ypbnv
Copy link
Contributor

@ypbnv ypbnv commented Jul 14, 2017

JIRA: https://jira.appcelerator.org/browse/TIMOB-24960

Description:
Strip the encoded output from line terminating symbols.

@ypbnv ypbnv added this to the 6.2.0 milestone Jul 14, 2017
@ypbnv ypbnv requested a review from garymathews July 14, 2017 14:49
Copy link
Contributor

@garymathews garymathews left a comment

Choose a reason for hiding this comment

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

CR: PASS

@ypbnv ypbnv modified the milestones: 6.1.2, 6.2.0 Jul 14, 2017
@ssjsamir ssjsamir self-requested a review July 14, 2017 16:18
Copy link
Contributor

@ssjsamir ssjsamir left a comment

Choose a reason for hiding this comment

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

FR Passed Was able to authenticate to HTTP clients without any issues.

Test Steps

  • Downloaded the SDK Build from this PR
  • Created a new alloy project
  • Copied the the test case from description https://jira.appcelerator.org/browse/TIMOB-24960
  • Ran the application
  • Pressed simple POST on the application
  • The callback was a success
  • Commented out the username and password which failed and uncommented out credentials which would pass in the test case
  • Pressed Basic Auth POST
  • The callback was a success
  • I then used credentials which were incorrect
  • Pressed Basic Auth POST
  • Callback was a failure as expected

Test Environment
Appcelerator Command-Line Interface, version 6.2.2
Google Nexus 6P (7.1.1)
Operating System Name: Mac OS X El Capitan
Operating System Version: 10.11.6
Node.js Version: 6.10.1
Xcode: 8.2
Appcelerator Studio: 4.9.0.201705251638

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants