Skip to content

[oauth2 cpp] add support to read credentials from file - #7606

Merged
wolfstudy merged 3 commits into
apache:masterfrom
jiazhai:issue_7486
Jul 28, 2020
Merged

[oauth2 cpp] add support to read credentials from file#7606
wolfstudy merged 3 commits into
apache:masterfrom
jiazhai:issue_7486

Conversation

@jiazhai

@jiazhai jiazhai commented Jul 20, 2020

Copy link
Copy Markdown
Member

Fixes #7486

Motivation

Add support to read credentials from file, make it align with java client.

Modifications

  • Add support to read credentials from file, make it align with java client.
  • add a test for it.

@codelipenghui

Copy link
Copy Markdown
Contributor

/pulsarbot run-failure-checks

@jiazhai

jiazhai commented Jul 23, 2020

Copy link
Copy Markdown
Member Author

/pulsarbot run-failure-checks

jiazhai added a commit that referenced this pull request Jul 24, 2020
### Motivation
Related with #7606, which add oauth2 authentication in cpp.
This PR is to add cpp client document for oauth2 authentication.

### Modifications

- Add documentation.
- rename file name "security-oauth" to "security-oauth2"
lbenc135 pushed a commit to lbenc135/pulsar that referenced this pull request Jul 25, 2020
### Motivation
Related with apache#7606, which add oauth2 authentication in cpp.
This PR is to add cpp client document for oauth2 authentication.

### Modifications

- Add documentation.
- rename file name "security-oauth" to "security-oauth2"

(cherry picked from commit 83e853b)
@wolfstudy

Copy link
Copy Markdown
Member

/pulsarbot run-failure-checks

@wolfstudy

Copy link
Copy Markdown
Member

/pulsarbot run-failure-checks

@wolfstudy

wolfstudy commented Jul 28, 2020

Copy link
Copy Markdown
Member

cpp ci keeps showing the following error:

---- Build Pulsar C++ client using image apachepulsar/pulsar-build:ubuntu-16.04 (pass <skip-clean> for incremental build)
pulsar-client-cpp/docker-build.sh: line 36: docker: command not found

@wolfstudy
wolfstudy merged commit 42d17c8 into apache:master Jul 28, 2020
wolfstudy added a commit that referenced this pull request Jul 29, 2020
### Motivation

Add support to read credentials from file, make it align with java client. 

### Modifications

- Add support to read credentials from file, make it align with java client. 
- add a test for it.

Co-authored-by: xiaolong.ran <rxl@apache.org>
(cherry picked from commit 42d17c8)
huangdx0726 pushed a commit to huangdx0726/pulsar that referenced this pull request Aug 24, 2020
### Motivation
Related with apache#7606, which add oauth2 authentication in cpp.
This PR is to add cpp client document for oauth2 authentication.

### Modifications

- Add documentation.
- rename file name "security-oauth" to "security-oauth2"
huangdx0726 pushed a commit to huangdx0726/pulsar that referenced this pull request Aug 24, 2020
### Motivation

Add support to read credentials from file, make it align with java client. 

### Modifications

- Add support to read credentials from file, make it align with java client. 
- add a test for it.

Co-authored-by: xiaolong.ran <rxl@apache.org>
lbenc135 pushed a commit to lbenc135/pulsar that referenced this pull request Sep 5, 2020
### Motivation

Add support to read credentials from file, make it align with java client. 

### Modifications

- Add support to read credentials from file, make it align with java client. 
- add a test for it.

Co-authored-by: xiaolong.ran <rxl@apache.org>
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.

Handle left work for cpp oauth2 client.

5 participants