Skip to content

[ZEPPELIN-5068] Angular username and password encoding#3923

Closed
Reamer wants to merge 2 commits intoapache:masterfrom
Reamer:angular_username_and_password_encoding
Closed

[ZEPPELIN-5068] Angular username and password encoding#3923
Reamer wants to merge 2 commits intoapache:masterfrom
Reamer:angular_username_and_password_encoding

Conversation

@Reamer
Copy link
Contributor

@Reamer Reamer commented Sep 24, 2020

What is this PR for?

This PR encodes username and password. Content-Type: application/x-www-form-urlencoded is set automatically.
I also found a small typo and corrected the spelling of Notebook.

What type of PR is it?

  • Bug Fix

What is the Jira issue?

How should this be tested?

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

@asfgit asfgit closed this in 2e69f71 Sep 28, 2020
asfgit pushed a commit that referenced this pull request Sep 28, 2020
### What is this PR for?
This PR encodes username and password. `Content-Type: application/x-www-form-urlencoded` is set automatically.
I also found a small typo and corrected the spelling of Notebook.

### What type of PR is it?
 - Bug Fix

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-5068

### How should this be tested?
* Travis-CI: https://travis-ci.org/github/Reamer/zeppelin/builds/729873263

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No

Author: Philipp Dallig <philipp.dallig@gmail.com>

Closes #3923 from Reamer/angular_username_and_password_encoding and squashes the following commits:

1ac7170 [Philipp Dallig] Correct spelling for Notebook
dd2f019 [Philipp Dallig] Encode username and password

(cherry picked from commit 2e69f71)
Signed-off-by: Philipp Dallig <philipp.dallig@gmail.com>
@Reamer Reamer deleted the angular_username_and_password_encoding branch September 28, 2020 14:37
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.

1 participant